feat(python): add local cross-encoder reranker support#445
Open
Hendrixx-RE wants to merge 3 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
All reported issues were addressed
You’re at about 96% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
Hendrixx-RE
force-pushed
the
feature/cross-encoder-reranker
branch
from
July 20, 2026 06:47
7000b7d to
01dfb22
Compare
Author
|
@HarshaNalluru if this is indeed the intended output,then Ill implement it on the js sdk also and wrap it up with tests and documentation |
Hendrixx-RE
force-pushed
the
feature/cross-encoder-reranker
branch
from
July 20, 2026 20:45
01dfb22 to
6f56b8a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Checklist
Please ensure that your PR meets the following requirements:
Description
I'm working on adding the cross-encoder option and have made the changes to the python sdk.
So far,Ive added a new QueryOptions class and removed the import from the precompiled rust core to add the flag and the top-k meathod.
This is still a work in progress as I have to:
-Update the JS sdk
-Update the elixir sdk
-Documentation
-Tests
Fixes #416
Type of Change