-
Notifications
You must be signed in to change notification settings - Fork 122
docs: Added more snippets to the hybrid search with cloud inference tutorial #1953
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
Conversation
✅ Deploy Preview for condescending-goldwasser-91acf0 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
ddcca7d
to
63c048d
Compare
Signed-off-by: Anush008 <[email protected]>
63c048d
to
4096dae
Compare
```csharp | ||
var denseModel = "sentence-transformers/all-minilm-l6-v2"; | ||
var bm25Model = "qdrant/bm25"; | ||
var dataset = LoadDataset("miriad/miriad-4.4M", "train[0:100]"); |
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.
The implementation of LoadDataset
is upto the user to keep the snippets concise.
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.
LGTM. I'm only wondering whether making a comment near to the loadDataset functions in the other languages would make sense. Something stating that it's a mock, and people can't fully reproduce it.
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.
Sorry, was reviewing two PRs at the same time and chose the wrong option here
Signed-off-by: Anush008 <[email protected]>
PREVIEW