Skip to content

feat(python): add local cross-encoder reranker support#445

Open
Hendrixx-RE wants to merge 3 commits into
usemoss:mainfrom
Hendrixx-RE:feature/cross-encoder-reranker
Open

feat(python): add local cross-encoder reranker support#445
Hendrixx-RE wants to merge 3 commits into
usemoss:mainfrom
Hendrixx-RE:feature/cross-encoder-reranker

Conversation

@Hendrixx-RE

@Hendrixx-RE Hendrixx-RE commented Jul 19, 2026

Copy link
Copy Markdown

Pull Request Checklist

Please ensure that your PR meets the following requirements:

  • I have read the CONTRIBUTING guide.
  • [] I have updated the documentation (if applicable).
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Review in cubic

@CLAassistant

CLAassistant commented Jul 19, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread sdks/python/sdk/src/moss/client/moss_client.py Outdated
Comment thread sdks/python/sdk/src/moss/client/moss_client.py Outdated
Comment thread sdks/python/sdk/src/moss/client/moss_client.py
Comment thread sdks/python/sdk/src/moss/client/moss_client.py Outdated
@Hendrixx-RE
Hendrixx-RE force-pushed the feature/cross-encoder-reranker branch from 7000b7d to 01dfb22 Compare July 20, 2026 06:47
@Hendrixx-RE

Copy link
Copy Markdown
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
Hendrixx-RE force-pushed the feature/cross-encoder-reranker branch from 01dfb22 to 6f56b8a Compare July 20, 2026 20:45
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.

Feature: Reranking support

2 participants