-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat: Add CohereRanker for 2.0 #7446
feat: Add CohereRanker for 2.0 #7446
Conversation
Pull Request Test Coverage Report for Build 8501345442Details
💛 - Coveralls |
Hi @anakin87 and @dfokina! Previously, I included I think I'm not adding cohere in the right place in |
Hey, @AnushreeBannadabhavi! Thank you for your contribution. Starting from Haystack 2.0, the integrations maintained by us live here: https://github.com/deepset-ai/haystack-core-integrations So, I would ask you to close this PR and open a similar one in the haystack-core-integrations repository, following these guidelines. Feel free to ask for clarification (if needed). |
Thanks @anakin87! Will open a PR in https://github.com/deepset-ai/haystack-core-integrations shortly |
I will close this one and wait for the other 💙 |
Related Issues
Proposed Changes:
Add
CohereRanker
for Haystack 2.0.Performs reranking of documents using Cohere reranking models. Reranks retrieved documents based on semantic relevance to a query.
For more information refer to Cohere reranker
How did you test it?
test_cohere.py
Notes for the reviewer
ApiError
with the messageinvalid request: list of documents must not be empty
.CohereRanker
since the exception raised by cohere is descriptive. However, I can add a check in theCohereRanker
itself to handle such a scenario.Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.