-
Notifications
You must be signed in to change notification settings - Fork 16
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
deps: upgrade llamaindex to 0.10.28 #316
Conversation
3a907d5
to
acd63f7
Compare
waiting for run-llama/llama_index#11896 to be released |
There's a conflict in transitive dependencies between langchain and llama-index...
|
note this is only for langchain dev, but yeah, we need that before merging this PR |
c4750a9
to
60c2c46
Compare
Update to 0.10.22 now, let's see if everything passes |
@@ -24,7 +24,7 @@ nemoguardrails = "^0.8.0" | |||
beautifulsoup4 = "^4" | |||
|
|||
llama-index = { git = "https://github.com/run-llama/llama_index.git", branch = "main" } | |||
llama-index-readers-llama-parse = { git = "https://github.com/run-llama/llama_index.git", branch = "main", subdirectory = "llama-index-integrations/readers/llama-index-readers-llama-parse" } | |||
llama-index-readers-llama-parse = { git = "https://github.com/nicoloboschi/llama_index.git", branch = "llama-parse", subdirectory = "llama-index-integrations/readers/llama-index-readers-llama-parse" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, I did the same PR then run-llama/llama_index#12236
so they reverted llama-index gemini upgrade with run-llama/llama_index#12126 ... let's wait for more clarity |
0763e7d
to
fcffcba
Compare
05f69af
to
22e3561
Compare
No description provided.