Skip to content

feat(constants): add $rerank stage#709

Open
DarshanaVenkatesh wants to merge 12 commits intomainfrom
COMPASS-10477-add-rerank-stage
Open

feat(constants): add $rerank stage#709
DarshanaVenkatesh wants to merge 12 commits intomainfrom
COMPASS-10477-add-rerank-stage

Conversation

@DarshanaVenkatesh
Copy link
Copy Markdown
Contributor

@DarshanaVenkatesh DarshanaVenkatesh commented Apr 27, 2026

Description

Add new $rerank stage to be used in aggregation pipeline

Open Questions

Checklist

Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
@DarshanaVenkatesh DarshanaVenkatesh marked this pull request as ready for review April 29, 2026 01:12
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 29, 2026

Coverage Status

coverage: 78.619%. remained the same — COMPASS-10477-add-rerank-stage into main

Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
Comment thread packages/mql-typescript/tests/stage/rerank.spec.ts Fixed
@DarshanaVenkatesh DarshanaVenkatesh force-pushed the COMPASS-10477-add-rerank-stage branch from aa65ee4 to 2b357b7 Compare April 30, 2026 20:48
@DarshanaVenkatesh
Copy link
Copy Markdown
Contributor Author

DarshanaVenkatesh commented Apr 30, 2026

I did not add a .spec typescript test as it needs $rerank to be defined in schema.d.ts to be meaningful. schema.d.ts is an output file that requires a change in mql-specifications. Will look into adding the change there and can add the test when we do the submodule bump in this repo.

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.

3 participants