Skip to content

Edit transformers requirement to be <4.35.0#2348

Closed
claudiatang-git wants to merge 3 commits intohuggingface:masterfrom
claudiatang-git:master
Closed

Edit transformers requirement to be <4.35.0#2348
claudiatang-git wants to merge 3 commits intohuggingface:masterfrom
claudiatang-git:master

Conversation

@claudiatang-git
Copy link
Copy Markdown

With safetensors serialization being the default in transformers v4.35.0, transformers v4.35.0 is currently not compatible with the sentence-transformers library as is. sentence-transformers cannot load safetensor models as of now, only pytorch_model.bin (pickle)

@tomaarsen
Copy link
Copy Markdown
Member

tomaarsen commented Nov 7, 2023

Hello!

SentenceTransformers can still load models with safetensors, so I don't think there is a need to restrict the maximum transformers version. See also #2345, which should improve the model loading such that loading a SentenceTransformer model no longer requires the full repository (potentially with multiple model files) to be downloaded.

  • Tom Aarsen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants