-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add modernbert-embed-large #146
Add modernbert-embed-large #146
Conversation
Hey @laulauland. Thanks for contributing. Could you please also add the model to the README.md? |
added! |
Thanks. LGTM. |
added the sample embeddings values and a separate test for modern bert |
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.
Thanks @laulauland. I believe the CI failure is unrelated to the change.
Also apologies for the delay. I have been OOO.
# [4.5.0](v4.4.0...v4.5.0) (2025-02-18) ### Features * Add modernbert-embed-large ([#146](#146)) ([8184799](8184799)) ## [4.5.0](v4.4.0...v4.5.0) (2025-02-18) ### 🍕 Features * Add modernbert-embed-large ([#146](#146)) ([8184799](8184799)) ### 🧑💻 Code Refactoring * organize the import and export of SparseModel ([#141](#141)) ([2f982f1](2f982f1)) ### 🧹 Chores * Bump hf-hub to 0.4.1 ([#142](#142)) ([b5911b4](b5911b4))
🎉 This PR is included in version 4.5.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
This adds ModernBERT-embed-large to the roster of available models.
ModernBERT is an encoder-only base model that was released at the end of last year. The model itself cannot be used for embeddings but a finetuned
-embed-*
version can be.