-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update requirements.txt #8
base: main
Are you sure you want to change the base?
Conversation
We had way too many dependencies which were probably sub dependencies, no need for us to list them. Signed-off-by: Chihurumnaya Ibiam <[email protected]>
Yes the all the langchain dependencies are proper as far as Pippy is concerned👍 |
I'll like to mention other dependencies which too are important to mention I think and have not been mentioned
These are quite fundamental ones used in running our RAG model and most of them are not there in our older version of requirements.txt hence I'll look forward to add them to with the langchain ones. |
If they're direct dependencies then we should add them, I know FAISS is one. |
Yes, this covers most of the fundamental dependencies ik. |
Alright, thank you! I'm still waiting for your input @XXXJumpingFrogXXX. |
I encountered one problem: when I try to load the model and tokenizer, the ModuleNotFoundError is raised. ModuleNotFoundError Traceback (most recent call last) /usr/local/lib/python3.10/dist-packages/unsloth/init.py in ModuleNotFoundError: No module named 'bitsandbytes' |
Hmmm, unsloth should've handled the installation of its own dependencies. It's also strange that you're experiencing this now when you've not experienced it before, you'll have to install bitsandbytes using pip. You should also use ticks to make code blocks easier to read, see code blocks. |
Installing bitsandbytes alone does not resolve the issue, as there are more packages waiting to be installed afterwards, such as triton, xformers, trl, etc. How should we modify requirements.txt to achieve the effects of the following two scripts? pip install unsloth |
By listing the packages passed as packages in requirements.txt. |
When I try to install the dependencies with our original requirements file, it will throw this error: The conflict is caused by: To fix this you could try to:
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts |
The missing packages might be numerous, making it difficult to manually add all the remaining required packages. |
Which is why you install the major packages and pip should handle the rest, can you start with a blank slate, install just the directly used dependencies and notice what other things get installed as they're being installed. |
We had way too many dependencies which were probably sub dependencies, no need for us to list them.
@kshitijdshah99 @XXXJumpingFrogXXX kindly review and confirm this is good.