Indexes only work if NetworkX ids are integers about gqlalchemy HOT 5 CLOSED

I see two ways to tackle this.

1. Add support for strings and escape the index when creating the cypher query with quotes.
2. Support all data types by hashing the ID with a function to an integer.

What do you think, is there a better way for approaching this?

from gqlalchemy.

Both options should be fine 😄 Option 1. seems a bit better to me because the same values are used directly (all types could be supported).

from gqlalchemy.

Yeah, I agree for strings, but what do you mean by "all types could be supported"? How would you support IDs of type "Person", for example? By using the hash approach that would be trivial.

Maybe we could support built-in types by escaping them so the representation stays the same, but if the type is unknown we could use the python built-in "id" function to get the id.

from gqlalchemy.

Thanks for reporting the issue, I agree with option one, this was overlooked in the initial implementation...

from gqlalchemy.

Here is a minimal example, example.py produces queries that can't be imported. Please rename the example files because .py and .dot can't be directly uploaded here 😄

example_dot.txt
example_py.txt
requirements.txt

from gqlalchemy.