-
Notifications
You must be signed in to change notification settings - Fork 613
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
Clarify what "More Resources" is by breaking it into its parts. #1238
Conversation
This also helps distinguish it from the "More Information" section. It adds the jinja variables `demo`, `paper`, and `repo`. See discussion at: https://huggingface.slack.com/archives/C025H22A3ND/p1669923069281609
The documentation is not available anymore as the PR was closed or merged. |
Hi @meg-huggingface thanks for opening the PR. I have read the related slack conversation. I am not against removing
By default (and for most model cards) the "other" section will not be displayed. But if the field is already used by some users, the Beside that, could you update |
I'll change to Other, but I do wonder if anyone is actually using the |
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.
Good to go !
This also helps distinguish it from the "More Information" section. It adds the jinja variables
demo
,paper
, andrepo
. See discussion at:https://huggingface.slack.com/archives/C025H22A3ND/p1669923069281609