Skip to content

Conversation

@TsuTikgiau
Copy link
Contributor

This MR is for registering the BAGEL model to the hub
https://huggingface.co/ByteDance-Seed/BAGEL-7B-MoT

Copy link
Contributor

@merveenoyan merveenoyan left a comment

Choose a reason for hiding this comment

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

@TsuTikgiau it would be great to add a minimal snippet to load the model as well 🤗

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.

Thanks for the quick turn around @TsuTikgiau - let's wait for @Wauplin to take a look as well!

countDownloads: `path_extension:"pth"`,
snippets: snippets.audioseal,
},
bagel: {
Copy link
Contributor

Choose a reason for hiding this comment

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

There are a lot other models w/ bagel tag: https://huggingface.co/models?other=bagel

maybe we can put bagel-mot or something?

cc: @Wauplin wdyt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe bagel-7b-mot

Copy link
Contributor

Choose a reason for hiding this comment

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

Sounds good to me!

Copy link
Contributor

Choose a reason for hiding this comment

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

actually on second thought bagel-mot might be better in case you release other variants of the model in the future, that you'd just need to put the same tag in the model readme and it'll just work.

Copy link
Member

Choose a reason for hiding this comment

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

Yes, I think bagel-mot is more future-proof.

Vaibhavs10 and others added 2 commits May 22, 2025 14:42
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.

@Vaibhavs10 Vaibhavs10 merged commit 170b542 into huggingface:main May 22, 2025
3 of 4 checks passed
@Wauplin
Copy link
Contributor

Wauplin commented May 27, 2025

Looks good! 🎉 Thanks for the bagel-mot change :)

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