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

[Falkor DB Integration Part 1] Implement FalkorGraphQueryEngine #37

Merged
merged 7 commits into from
Oct 7, 2024

Conversation

randombet
Copy link
Collaborator

@randombet randombet commented Sep 9, 2024

Why are these changes needed?

This is the first part of Falkor DB integration.
In this PR, I wrap up implement FalkorGraphQueryEngine and setup the integration test.

In the next few PRs, I will implement Falkor DB Graph RAG capability and add a notebook example.

Related issue number

GraphRag Integration

Checks

@randombet randombet changed the title [NOT Ready for review] Implement FalkorGraphQueryEngine [Falkor DB Integration Part 1] Implement FalkorGraphQueryEngine Sep 9, 2024
@randombet randombet marked this pull request as ready for review September 9, 2024 22:40
Copy link
Collaborator

@Hk669 Hk669 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@randombet can we have a reference notebook for this PR?

from graphrag_sdk import KnowledgeGraph, Source
from graphrag_sdk.schema import Schema

sys.path.append(os.path.join(os.path.dirname(__file__), "../../.."))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'i think this isnt necessary.

@randombet
Copy link
Collaborator Author

@randombet can we have a reference notebook for this PR?

@Hk669 The notebook will be included in the next PR when we add the capability to an agent.

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.

3 participants