Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom step #85

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Custom step #85

wants to merge 2 commits into from

Conversation

kefirbandi
Copy link
Contributor

James,

this is the functionality I mentioned before.
This allows the usage of node.custom_step('friend') or `node.custom_step('out.out').

I made a few changes, which I believe are rather independent from the implementation details of lower layers. I send them in different pull requests.

@espeed
Copy link
Owner

espeed commented Apr 24, 2013

Thanks. I'll look at this more when I have some time...not sure I understand the use case here. You can already do a custom step (one that's not stored in a .groovy file) via:

>>> g.gremlin.query("g.v.(id).out.out", {id: 1})

@kefirbandi
Copy link
Contributor Author

I agree. This is a convenience feature.
It comes especially handy in cases when you don't have direct access to "g".
For example in the class definition of a model, where you now have to write something like

a=self._client.gremlin("g.v(id).out.out", {'id': self.eid})
b=initialize_elements(self._client,a)

Instead it is simpler to write

b=self.custom_step("out.out")

This and the other pull requests are some features which I use, so I thought I share them with you. I am happy if these (or some variations) become part of the main line (like #90), but is absolutely no problem for me if for any reason you have a different opinion.

@espeed
Copy link
Owner

espeed commented May 22, 2013

A similar method existed in Bulbs 0.2 and I removed it to simply the code in Bulbs 0.3 when I reworked the Gremlin module:

https://github.com/espeed/bulbs/blob/0.2.x/bulbs/element.py#L143

Let's hold off on this for now -- I have an ideas for how to simply Gremlin calls from Models that may make this approach unnecessary .

@FAHADXK88
Copy link

من سياسة الاعتراف عن اعتراف التركيبات في الحمامات من سجلات البورسلان على نظام الأسمنت في سجل الديكورات المنظمة في تنظيم كهربائي في سجلات الاشتعال عن عوامل الجينات في جيني الذي يحمل على تنظيم المتغيرات عن ميكنة الشور في سجل الحنفية عن تحقيقات الدوش في سجل الشطاف عن تشطيف التغريق في سباحة الإنسان عن تعامل التوقيت في استراتيجيات الأعمال الجنسية عن تجريبات المجرد عن سواعد الأعمال المؤكد في توكيلات التأكيد عن بناية الأفراح من سجلات الأكبر عن شركات التوكيل من صادوه عن صاهوديات اليهود في سجل الهوتميل
عن قياس الاعتماد المحرر عن حرارة المياه في شتاء الصيف عن صيف البرودة في مشلتت الشاطرين في زعيم الملاحق عن منارات الامارات في سجل العين عن ابو ظبي في فطيرة البناء عن اربعمائة شالية في ثلاثة فنادق بحرية بحور الميناء المراكز في سعر التكلفة 2 مليارات درهم بقياسات اللورد عن شيخ الفجيرة من عوامل الاشتراكات الرئيسية عن توثيقات الوثائق في شاليهات الفجيرة بمستند المزارع الرئيسي في مجلة التشهير عن كيفك في سجل العشر ملايين دينار بمباريات الشراء عن تشاورهم في سجلي وانا بوطواط الفراش غير محدثات المعروف في اسمافيات في سجل البنوك عن توك من من سيارة بيوك عن توبك في سيارة فيراري عن الله اكبر كبيرا والحمدلله كثيرا .... في انقسامات الحوامل الحدودية عن حدود حاملات الطائرات في لعبة ريسك الدولية عن العابي المسلمة في سجل الفلسطينيين عن لعبتي اليهودية في سجل حماس بنهج إيراني عن سجل فتح في ابتكارات اخواني ز ذلك عن قياس البعوض في سجل الناموس عن موس الحلاقة في تسجيل المسجل بأنقسام التال من أبيب عن بيانات سعادة الرئيس نتنياهو المحترم والموقر عن إيقاف النار للأخوة الفلسطينيين عن قسطنطينية التواريخ عن مؤرخين البارات في سجل الدجل عن صحاري في سياسة المغتربين عن بيانات التعريف في سياسة التعريف عن عرف التغريدات في تويتر من اكسايتات عن اكس التجديد في تعاين التلاعب من جد و جديد وكذلك نفعل في سعيد بعيده ومسيعيد المسعده عن هلا بالروح من ريه وكل الناس هاريه.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants