search

onuratakan / gpt-computer-assistant

gpt-4o for windows, macos and linux
MIT License
4.75k stars 441 forks source link

Pin dependencies version #73

Closed vietnguyengit closed 4 weeks ago

vietnguyengit commented 4 weeks ago

Unpinned dependency versions cause conflicts, and dependencies should not be left unpinned in the requirements.txt file either. Feel free to change the versions as you prefer, but please avoid leaving them unpinned.

You can consider using poetry for better dependency management. It provides a lock file and stores the version of every single dependency, including dependencies of other dependencies. Poetry's dependency resolver is better than pip's as well. You can avoid issue like this: https://github.com/onuratakan/gpt-computer-assistant/issues/60

I can submit a PR to switch to using poetry; however, since it's your app, I won't touch the architecture unless requested. For now, this PR aims to fix immediate problem.

I also modified the .gitignore because I and probably many others using JetBrains IDEs.

Error logs using the default requirements.txt

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
gpt-computer-assistant 0.0.0 requires langchain==0.2.3, but you have langchain 0.1.20 which is incompatible.
gpt-computer-assistant 0.0.0 requires langchain-community==0.2.3, but you have langchain-community 0.0.38 which is incompatible.
gpt-computer-assistant 0.0.0 requires langchain-experimental==0.0.60, but you have langchain-experimental 0.0.58 which is incompatible.
gpt-computer-assistant 0.0.0 requires langchain-openai<0.2.0,>=0.1.8, but you have langchain-openai 0.1.7 which is incompatible.
gpt-computer-assistant 0.0.0 requires langgraph<0.0.65,>=0.0.64, but you have langgraph 0.0.51 which is incompatible.

Cheers

onuratakan commented 4 weeks ago

Hi, you are so right and thank you so much. If you are available can i want your poetry experience.

vietnguyengit commented 4 weeks ago

Hi, you are so right and thank you so much. If you are available can i want your poetry experience.

Thank you for your amazing work into this project @onuratakan

I'll submit the poetry PR then. I've created an issue here for traceability: https://github.com/onuratakan/gpt-computer-assistant/issues/75

If you could assign me to it, it would help avoid redundant efforts.

Meanwhile, if you could review this PR for an immediate fix because it doesn't work out-of-the-box.

Thank you.