-
Notifications
You must be signed in to change notification settings - Fork 802
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 Static Embeddings blog #2598
Conversation
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.
Reads very nicely @tomaarsen - very detailed, I left some suggestions to hopefully increase the impact of the blogpost 🤗
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.
Amazing! In addition to the suggestions below (mostly nits), I think we could link to the previous blog posts somewhere: https://huggingface.co/blog/train-sentence-transformers, https://huggingface.co/blog/matryoshka.
I think we can also reword the conclusion a bit to focus more on the method than the models (similar to what I suggested for the intro). I can suggest something later if you agree.
I modified some, here and there Co-authored-by: Pedro Cuenca <[email protected]>
Co-authored-by: Pedro Cuenca <[email protected]>
Co-authored-by: Pedro Cuenca <[email protected]>
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.
Nice! Thanks a lot for factoring in changes @tomaarsen
Preparing the Article
md
file. You can also specifyguest
ororg
for the authors.