Mention no images added to repository#14738
Conversation
| example. | ||
| 7. Due to the rapidly growing repository, it is important to make sure that no file that would significantly weigh down | ||
| the repository are added. This includes images, videos and other non-text files. We prefer to leverage a hf.co hosted `dataset` like | ||
| the ones hosted on [`hf-internal-testing`](https://huggingface.co/hf-internal-testing) in which to place these files and reference |
There was a problem hiding this comment.
FYI the repo I use for documentation images is called "documentation-images": https://huggingface.co/datasets/huggingface/documentation-images
sgugger
left a comment
There was a problem hiding this comment.
Thanks for adding this note. As said on another PR, I think it would be more efficient to couple this with a migration of existing images to the same dataset and completely remove the imgs folder, this way no one will be tempted to add an image.
Co-authored-by: NielsRogge <48327001+NielsRogge@users.noreply.github.com>
|
WIll merge this as a first step! |
|
Merged too soon? We need a concrete destination for images that we all share. |
|
but it's not part of this PR that was merged. i.e. @NielsRogge's suggestion wasn't integrated into the doc. additionally please please keep these instructions in |
Mention that images shouldn't be added to the repository as it will otherwise significantly weigh down the repository.