upgrade graphrag sample to v2.3+#6744
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