Skip to content
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

[Feature Request]: Upgrade llama-index-embeddings-text-embeddings-inference package dependency requirements #14163

Closed
Avinash-Raj opened this issue Jun 14, 2024 · 0 comments · Fixed by #14213
Labels
enhancement New feature or request triage Issue needs to be triaged/prioritized

Comments

@Avinash-Raj
Copy link

Feature Description

As for now llama-index-embeddings-text-embeddings-inference supports

llama-index-embeddings-huggingface = "^0.1.1"

but in the recent times, the above package has the latest version as 0.2.1 which won't be supported by the above inference package because of the above pinned version on requirements. Can we change the above dependency to support llama-index-embeddings-huggingface == 0.2.1 ?

Reason

With respect to the recent change in transformers package, I want llama-index-embeddings-huggingface to be upgraded to the latest ie. 0.2.1 otherwise I might endup in below error.

TypeError: BertModel.__init__() got an unexpected keyword argument 'safe_serialization'

Value of Feature

No response

@Avinash-Raj Avinash-Raj added enhancement New feature or request triage Issue needs to be triaged/prioritized labels Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triage Issue needs to be triaged/prioritized
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant