-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
DRAFT: Add Graph store and Graph QA #2514
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Will have a look at this tomorrow! Apologies for the delay. |
@jacoblee93 no worries, I really didn't do too much, just did a very early PR to get your feedback and make sure I'm going in the right direction before I put more time. |
Hey @gkorland really sorry for the delay - had a chat with @baskaryan and I don't think we have a clear idea of the right common interface between graph stores at the moment - would you be up for implementing it like you did in Python or extending a very minimal base class? I think adding some |
I pushed the other day a simple interface for GraphStore see: langchain-ai/langchain#11012 |
@jacoblee93 I noticed the PRs #2741 #2749 were merged so I guess there is no need for this PR, I'll submit a new PR based on these PRs |
@jacoblee93 see #2774 |
An initial PR just to see that I'm building the right API.
Fixes #2383