-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
[ENH] Document Embedding: add SBERT #839
Conversation
Codecov Report
@@ Coverage Diff @@
## master #839 +/- ##
==========================================
+ Coverage 76.92% 76.96% +0.03%
==========================================
Files 85 85
Lines 11966 11950 -16
Branches 1880 1877 -3
==========================================
- Hits 9205 9197 -8
+ Misses 2454 2449 -5
+ Partials 307 304 -3 |
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.
Thank you for implementing it looks good. I just dropped a few minor comments.
Please also rebase this PR to master (sorry I was changing callbacks already since I didn't notice that you have been changing that here).
Before it can get merged we need to add GPU sbert embedders to the server. Hope it will be done soon. :)
402e42c
to
76457e2
Compare
76457e2
to
b67016c
Compare
b67016c
to
9df9fff
Compare
9df9fff
to
170cd4c
Compare
Issue
Implements #639.
Description of changes
Added multilingual SBERT into Document Embedding widget.
Includes