-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Integrate with Hugging Face #128
Conversation
Is this also in the to-do list at huggingface/transformers#32394 ? |
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.
@NielsRogge Thanks for supporting ckpt download from HF. Can we make huggingface_hub
a soft dependency?
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 for adding support to load the models through HF hub.
This is great! Is there a plan to make it compatible with transformers, like SAM 1? https://huggingface.co/docs/transformers/en/tasks/mask_generation |
Yes there is, the authors are adding a PR here: huggingface/transformers#32394 |
Fantastic! Can't wait for the PR to be merged! |
This is just amazing <3 |
Sorry potentially dumb question, but should this line in the video predictor be changed from |
Integrate with Hugging Face
This PR fixes #1. It aims to make it possible to load any SAM 2 model directly from the hub.
The checkpoints are already part of the facebook org:
This PR allows to do the following:
This also ensures that download stats work, meaning you'll be able to see how many times people actually load one of these checkpoints from the hub. This will also allow us to compare to the download stats of Transformers which will be interesting :)
To do: