-
Notifications
You must be signed in to change notification settings - Fork 337
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
Add Cohere Rerank 3 Support #466
base: main
Are you sure you want to change the base?
Conversation
@bigadsoleiman @azaylamba -- Do you have any ideas why these changes of adding Cohere Rerank 3 support aren't working? When I click on the I made sure to define the In the browser dev tools, I can see the array only has the one cross-encoder:
|
Does your |
Is the following
This happened after I re-ran Re-running everything did make the cohere rerank model show up in the config file, but now I'm having this new issue. |
@ystoneman Can you share the config.json file? |
Thanks for following up @azaylamba. Here's my config.json:
|
Hi @ystoneman, this issue is due to some resource that was included in the nag-suppression rules to not be present any more. In particular |
Hi @massi-ang, thanks for your response. I don't think I'm disabling the SageMaker cross-encoder, because both cross-encoders are specified in the config.json. My desired behavior is that the SageMaker cross-encoder still gets deployed, but I want to provide the ability to toggle between that and the Cohere rerank-english-v3.0 external API endpoint in the cross-encoder dropdown in the UI. Could you please clarify if there's a better way to handle this scenario in the suppression rules? |
Issue #, if available: #280
Description of changes: Currently, this solution only supports cross-encoder/ms-marco-MiniLM-L-12-v2. I want to add Cohere Rerank 3 as an option.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.