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

Add Gremlin graph storage #494

Merged
merged 2 commits into from
Dec 20, 2024
Merged

Add Gremlin graph storage #494

merged 2 commits into from
Dec 20, 2024

Conversation

alllexx88
Copy link
Contributor

Add Gremlin graph storage.

Note that at least my setup suffered from an anyio issue, that has been fixed a couple of days ago, with commit agronholm/anyio@c518300 so I had to install it from git before the next version is released.

@alllexx88
Copy link
Contributor Author

I see the linting error report, due to logging.basicConfig(format="%(levelname)s:%(message)s", level=logging.WARN) line before the lightrag imports. I can fix it, but setting logging level with this line doesn't change the loglevel when issued after lightrag imports.

@LarFii
Copy link
Collaborator

LarFii commented Dec 20, 2024

Thank you for your contribution! To avoid the linting error during , you can temporarily comment out this line. Additionally, you can add a comment to describe its functionality.

@alllexx88
Copy link
Contributor Author

Thanks @LarFii ! Done together with one little fix (using await asyncio.gather instead of consecutive awaits in get_node_edges())

@LarFii
Copy link
Collaborator

LarFii commented Dec 20, 2024

Thanks!

@LarFii LarFii merged commit 4e6f3bb into HKUDS:main Dec 20, 2024
1 check passed
@alllexx88 alllexx88 deleted the gremlin_graph branch December 20, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants