-
Notifications
You must be signed in to change notification settings - Fork 472
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
Update cross-encoder model prefix #6686
Conversation
Signed-off-by: Fanit Kolchina <[email protected]>
@HenryL27 Could you please review this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! one nit, especially in case we someday add a model that doesn't start huggingface/cross-encoders
@@ -66,12 +66,12 @@ The following table provides a list of sparse encoding models and artifact links | |||
|
|||
Cross-encoder models support query reranking. | |||
|
|||
The following table provides a list of cross-encoder models and artifact links you can use to download them. | |||
The following table provides a list of cross-encoder models and artifact links you can use to download them. Note that you must prefix the model name with `huggingface/cross-encoders`, as shown in the **Model name** column. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that you must prefix the model name with
huggingface/cross-encoders
, as shown in the Model name column.
idk if this sentence is necessary since it's documented in the table. As a user I copy/paste the model name out of the table and stick it in the pretrained model _register body, so as long as the thing in the table is correct, I'm happy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have the same sentence for transformer models. Mainly because when you look at the model config, there is a name
field that does not include the full prefix (for example, cross-encoder/ms-marco-MiniLM-L-6-v2
). So users might be confused why the name
in the table is not the same as the name
in the model config.
Signed-off-by: Fanit Kolchina <[email protected]> (cherry picked from commit 7cac2a7) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 7cac2a7) Signed-off-by: Fanit Kolchina <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Closes #6658
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.