-
Notifications
You must be signed in to change notification settings - Fork 158
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
stellasia
left a comment
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
rankerandalpha, 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
naivewhich 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):