-
Notifications
You must be signed in to change notification settings - Fork 98
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 request] Support latest version of networkx #55
Comments
edit: I'm sorry, the real problem seems to be gruut
|
I've already opened rhasspy/gruut#48 there to add Numpy 2 support and can extend it to networkx. Hope it will get merged. |
@ic-mw gruut 2.4.0 was released and removed the upper bounds for numpy and networkx. However, coqui-tts 0.24.1 still has the old gruut version pinned. I opened #56 to update it, but there are still a few remaining issues with Numpy 2 support in dependencies. I'll wait a bit longer before merging to see if they get resolved soon, there are a few other updates I want to make before the next coqui-tts release anyway. But feel free to install from that branch in the meantime. |
excellent, thank you. we'll try out the latest changes. |
This is merged into the dev branch now and I plan to make a new release soon as well. |
🚀 Feature Description
I am working on a project which uses llama-index-core, but it depends on a version of networkx which is incompatible with coqui-tts.
and llama-index-core (0.10.51) depends on networkx (>=3.0), llama-index-core (>=0.10.37.post1,<0.11.0) requires networkx (>=3.0). And because llama-index-llms-azure-openai (>=0.1.5,<0.2.0) requires networkx (>=3.0) or llama-index-core (>=0.10.37.post1,<0.11.0) (5), llama-index-llms-azure-openai (>=0.1.5,<0.2.0) requires networkx (>=3.0) And because coqui-tts (>=0.24.1,<0.25.0) requires networkx (>=2.5.0,<3.0.0) (1),
Solution
Could you please consider supporting the current version of networkx?
The text was updated successfully, but these errors were encountered: