-
Notifications
You must be signed in to change notification settings - Fork 108
Add linear hybrid search ranker #284
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
@CodiumAI-Agent /update_changelog |
Changelog updates: 🔄 2025-02-24Added
|
b0914d6
to
533afc6
Compare
7bf434a
to
d8205cc
Compare
98e8faf
to
4fe39b5
Compare
Thanks for having this feature implemented! Nicely done :D |
Changelog updates: 🔄 2025-02-25Added
|
b47b6b3
to
6a923d5
Compare
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 work, thanks for addressing this issue!
Changelog updates: 🔄 2025-02-26Added
|
Description
This PR is a follow-up work from a previous PR to rework the Hybrid search ranking step. This PR allows users of the HybridRetriever and HybridCypherRetriever to specify
ranker
andalpha
, which ranks the relevance of the retrieved results over the vector index and fulltext index search using the score:In practice, the vector index score is multiplied by alpha and fulltext index score is multiplied by 1 - alpha, then they are compared and ranked.
The default used is still
naive
which is the naive implementation of comparing normalized scores.The pros and cons of using this ranker:
Pros
alpha
.Cons
Despite these tradeoffs, I think this is valuable for users as it is an upgrade over the naive ranker.
Type of Change
Complexity
Complexity: Medium
How Has This Been Tested?
Checklist
The following requirements should have been met (depending on the changes in the branch):