[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/340 How do we use this again? This is just for the `GeoPoint`-style adding classes? Can you provide an example so we can add it to the docs. See: https://github.com/apache/tinkerpop/blob/TINKER

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
Github user leifurhauks commented on the issue: https://github.com/apache/tinkerpop/pull/340 > This is just for the GeoPoint-style adding classes? More generally, it can be used to create expressions that make use of APIs, presumably provided by the graph vendor, that aren't s

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
Github user leifurhauks commented on the issue: https://github.com/apache/tinkerpop/pull/340 @okram , I realized the current implementation of bindings as dicts is breaking 2/3 compatibility. It also makes it bit hairy to extract just the key in `RawExpression`. I would sugg

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/340 Please. Use tuple though instead of introducing a new class. Thanks. > On Jun 16, 2016, at 1:54 PM, Leifur Halldor Asgeirsson wrote: > > @okram

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/340 @leifurhauks do you know why travis is not happy? the build took some errors it seems. Can you please look into that as you push your next change? --- If your project is set up for it, you can

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
Github user leifurhauks commented on the issue: https://github.com/apache/tinkerpop/pull/340 I believe the failures are due to the fact that the aenum package (introduced in 8ee2904 ) is not installed in the CI environment. As a result, the gremlin_python module errors on import.

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/340 ah - didn't realize that build was hosed in the TINKERPOP-1278 branch itself. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I