Feat/llama index version update #142
Conversation
…Zipstack/unstract-sdk into feat/llama-index-version-update
Signed-off-by: Praveen Kumar <[email protected]>
pk-zipstack
requested review from
Deepak-Kesavan,
chandrasekharan-zipstack and
gaya3-zipstack
Co-authored-by: Chandrasekharan M <[email protected]> Signed-off-by: Praveen Kumar <[email protected]>
Co-authored-by: Chandrasekharan M <[email protected]> Signed-off-by: Praveen Kumar <[email protected]>
Co-authored-by: Chandrasekharan M <[email protected]> Signed-off-by: Praveen Kumar <[email protected]>
Co-authored-by: Chandrasekharan M <[email protected]> Signed-off-by: Praveen Kumar <[email protected]>
| "llama-index-vector-stores-qdrant==0.4.2", | ||
| "llama-index-llms-openai==0.3.12", | ||
| "llama-index-llms-palm==0.3.0", | ||
| "llama-index-llms-mistralai==0.3.1", | ||
| #Todo: Remove the version locking for mistralai below | ||
| #once the the following error is resolved from mistralai side | ||
| #Unable to import llm adapters : No module named 'mistralai.models.chat_completion' |
There was a problem hiding this comment.
@pk-zipstack Hope this mistral error is not occuring with 1.2.5. Earlier we had hard-pinned to 0.4.2 because of thie error as mentioned in ToDo.
There was a problem hiding this comment.
@gaya3-zipstack This error is not occuring with 1.2.5. Can I go ahead and remove the comments?
| @@ -35,7 +35,7 @@ def parse_llm_err(e: Exception, llm_adapter: LLMAdapter) -> LLMError: | |||
| err = OpenAILLM.parse_llm_err(e) | |||
| elif isinstance(e, AnthropicAPIError): | |||
| err = AnthropicLLM.parse_llm_err(e) | |||
| elif isinstance(e, MistralException): | |||
| elif isinstance(e, MistralError): | |||
There was a problem hiding this comment.
Hope we did a static check on mistralai.exceptions and mistralai.models to see that MistralError now includes MistralException.
There was a problem hiding this comment.
@gaya3-zipstack I checked the latest release notes and ran a static analysis on mistralai.exceptions and mistralai.models.SDKError(MistralError) now replaces MistralException and handles the same errors, ensuring compatibility.
There was a problem hiding this comment.
llama-index is also directly used as a dependency in OSS
Hope there will be a corresponding OSS PR to take care of these other repos where we use it.
Ideally, we will need to remove all this. We need to pick it up soemtime.
Pls add it as a technical debt by creating a JIRA for the same @pk-zipstack
What
MistralExceptionwithSDKErrorfor the updatedmistralai 1.2.5dependency.Why
How
MistralExceptionwithSDKError.Related Issues or PRs
Dependencies Versions / Env Variables
Notes on Testing
single-pass extraction,summarizer,LLMChallengeretc..Screenshots
Checklist
I have read and understood the Contribution Guidelines.