Skip to content

Update checklist#1658

Merged
Vaibhavs10 merged 4 commits intomainfrom
update_checklist
Mar 31, 2025
Merged

Update checklist#1658
Vaibhavs10 merged 4 commits intomainfrom
update_checklist

Conversation

@NielsRogge
Copy link
Contributor

This PR adds a link to the uploading models best practices guide to the model release checklist.

@NielsRogge NielsRogge requested a review from Vaibhavs10 March 31, 2025 12:51
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Contributor

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small suggestion to increase the readability of the doc

NielsRogge and others added 2 commits March 31, 2025 15:20
Co-authored-by: vb <vaibhavs10@gmail.com>
@Vaibhavs10 Vaibhavs10 merged commit 7813634 into main Mar 31, 2025
2 checks passed
@Vaibhavs10 Vaibhavs10 deleted the update_checklist branch March 31, 2025 13:41
@julien-c
Copy link
Member

let's wait for a @gary149 review before merging as he was quite intentional about what to mention/in which order


When uploading models to the hub, it's recommended to follow a set best practices:

- push weights to separate model repositories. Example: prefer uploading individual quantizations/precisions in a standalone repo like [this](https://huggingface.co/jameslahm/yolov10n) over all types/versions in one like [this](https://huggingface.co/kadirnar/Yolov10/tree/main).
Copy link
Contributor

@gary149 gary149 Mar 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would look better if the list was capitalized and more consistant

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will submit a follow-up PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nvm just saw #1660, I assume that resolves it

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.

5 participants