Skip to content
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

Allow None as query for BM25 and hybrid #924

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Allow None as query for BM25 and hybrid #924

merged 1 commit into from
Feb 29, 2024

Conversation

dirkkul
Copy link
Collaborator

@dirkkul dirkkul commented Feb 29, 2024

Closes #894

For Bm25:
If query is None, => normal GET, eg results have all score 0 (if you request them)
For hybrid:
if query is None, vector is not None, alpha => 1, pure vector search, but still hybrid-type scores (scaled)
if vector is None, query is not None, alpha = 0, pure keyword search, but still hybrid-type scores (scaled)
both None => normal GET, eg results have all score 0 (if you request them)

@dirkkul dirkkul merged commit b06a840 into main Feb 29, 2024
35 checks passed
@dirkkul dirkkul deleted the none_queries branch February 29, 2024 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accept None as query to make complex usage easier
2 participants