-
Notifications
You must be signed in to change notification settings - Fork 7.8k
upgrade graphrag sample to v2.3+ #6744
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
Conversation
|
@ekzhu , Graphrag requires an update to opena > 1.68 where the assistants api is deprecated in favour of the responses API. Should we remove/deprecate the assistant agent given that openai does not really support that anymore? |
Can we move the extra for open ai assistant to its own "openai-assistant" with the older version as the upper, and then update "graphrag" extra with its required open ai version? |
|
It will be a breaking change and require anyone using both OpenAI assistant and GraphRag extension to upgrade |
|
Is this issue resolved already, or is there still work going on? I can't use GraphRAG's latest version as a tool for the AutoGen agent. I am getting the error ModuleNotFoundError: No module named 'graphrag.config.config_file_loader'. I can use its older version, like 1.2.0 or something, but that also has issues of its own, which is understandable as it is an older version. I really need to use the latest version of GraphRAG with the AutoGen agent. Is there any way I can do this right now? |
We need to get this PR through for this to be possible. |
|
@victordibia now let's put an OpenAI version constraint in the graph rag extra UPDATE: note needed. The graph rag extra already takes care of the constraint. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6744 +/- ##
==========================================
+ Coverage 79.96% 80.74% +0.77%
==========================================
Files 235 234 -1
Lines 18184 17892 -292
==========================================
- Hits 14541 14447 -94
+ Misses 3643 3445 -198
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
The primary issue has been resolved now. LocalSearchTool and GlobalSearchTool can now be imported easily. However, I think you need to modify the _local_search.py file a little bit. The issue now is that when initializing the GlobalSearchTool and LocalSearchTool using the .from_settings() function, we can initialize GlobalSearchTool but not LocalSearchTool. It is giving the error that a 'str' object has no attribute .resolve(). I have checked the code of both _local_search.py and _global_search.py and found that the .from_settings() function is not converting the str into a Path, unlike what it does in _global_search.py. Therefore, by just adding: if isinstance(root_dir, str): the issue is fixed, and we can finally use both LocalSearch and GlobalSearch in the autogen agent. |
Why are these changes needed?
upgrade graphrag sample to v2.3+
Related issue number
Closes #6210
Closes #6201
Checks