-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
ingest.py - versioning #56
Comments
What version of the main are you running? We changed runners to modules inside ./casalioy So your env at least should be listing those. also you are likely missing /casalioy/ask_libgen.py. We had an issue with a PR so I had to revoke some earlier changes. Besides GUI this main's version should be stable |
I've cloned the most recent main commit and I'm running through the setup now. I'll let you know how it goes. |
Oh feel free to PR such things and I'll commit ASAP. Are you running fine again? |
Will do! I'm just heading home and will take a look at it once I get back. "python -m pip install --force streamlit sentence_transformers" is taking quite some time to run, so I'll hopefully have some good news once I am home 👍🏻 |
Jumping in late: you were just missing |
Giving this another shot this morning. The installation completed, I modified my .env to the following before attempting to run ./casalioy/ingest.py:
I was then hit with this error:
I cloned the all-MiniLM-L6-v2 to my root CASALIOY directory and matched my .env file to that in the README.md and ended up with this:
Am I missing something super simple here? I ran the git commands suggested in the last error and was hit with:
|
Ok, I think I may have sorted it. I addressed the following issue by referring to this thread:
I installed git-lfs and then ran "git lfs pull" in the "all-MiniLM-L6-v2" repo I pulled from HuggingFace. Running ingest.py now gives me:
The README..md file definitely needs updating and I'll see if I can get to it later this afternoon. |
I'm not sure I follow, isn't that a git-fls issue ? (which isn't used in the readme, nor required) PS: the part of the README about downloading models will be gone when #61 is merged Edit: I just reread your original problem: you don't actually mentioning using git lfs anywhere in the first place, so maybe it's just the error message that mislead you. There's no need to use it. |
Beat me to it, I saw your edits as I was typing out my response about the models in the README. |
Note for people who have the same issue: the actual problem you had comes from this line:
This is because you're using (as in the README) the old format q4 instead of q5. We'll adjust the readme. |
I stumbled on the same thread and re-edited my comment before closing this issue. Everything is running smooth and ingestion is now super-fast. Keep up the awesome work! |
I'm suddenly running into an issue when running ingest.py where I am being flagged with this error instead of the script processing as it should:
I am running CASALIOY through WSL on Ubuntu 22.04.2LS.
I was able to successfully run the ingestion script this morning against a 5mb PDF and the results were pretty good. I updated my repo to the latest version and I am now getting this error, despite rebuilding venv and running through the installation instructions to be on the safe side.
The text was updated successfully, but these errors were encountered: