-
Notifications
You must be signed in to change notification settings - Fork 324
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
feat: fastembed integration #268
feat: fastembed integration #268
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.
Still need to run locally, looks pretty solid and straightforward
- imo
rig-fastembed
as the example name is confusing, can it be renamed to likelocal_embeddings
orvector_search
, etc. - update README.md w/ new logo
I agree with the SparseModel
suggestion
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.
Works like a charm, pretty awesome crate 🔥.
Mergable after failing tests.
Merging as the tests all pass |
Closes #228
We may want to add other things like
SparseModel
in the future. However, this PR aims to serve as a starting point for the integration so this is somewhat out of scope (for the PR).