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

Feature/abstract rag interface #59

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

hbjugo
Copy link
Collaborator

@hbjugo hbjugo commented Jan 31, 2025

Almost finished adapting graphrag for MMORE.

Some things are still left to do, in particular regarding the polymorphism between the two retrievers which is why I tagged @paultltc.

Right now, the RAGConfig has a RetrieverConfig attribute, but I don't really see how to change it so we don't have to manually change between the GraphRAGGlobalRetrieverConfig and RegularRetrieverConfig each time. Any idea is welcome :)

I'm not too aware of naming conventions for the project as well, so feel free to hmu with any changes that I should make for this.

I also included the various examples .yaml file, along with a sample of the ICD11 (only the first 10% dataset).

To index, I added an -i option to the CLI command, to indicate whether to use the regular or graphrag indexer. HMU if anything is not clear !

@hbjugo hbjugo requested review from lighthea and paultltc January 31, 2025 16:09
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.

1 participant