Skip to content

update longcat-video-avatar-1.5 to model libraries#2199

Open
lile839 wants to merge 1 commit into
huggingface:mainfrom
lile839:main
Open

update longcat-video-avatar-1.5 to model libraries#2199
lile839 wants to merge 1 commit into
huggingface:mainfrom
lile839:main

Conversation

@lile839
Copy link
Copy Markdown

@lile839 lile839 commented May 28, 2026

update longcat-video-avatar-1.5 to model libraries


Note

Low Risk
Single registry entry with no runtime or security-sensitive logic changes.

Overview
Registers LongCat-Video-Avatar-1.5 as a Hub modeling library in MODEL_LIBRARIES_UI_ELEMENTS, keyed by longcat-video-avatar-1.5 (matching library_name on models).

The entry points to the LongCat-Video repo, keeps the library off the main models filter (filter: false), and defines download counting via .safetensors files or config.json.

Reviewed by Cursor Bugbot for commit ac18cb1. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor

@Wauplin Wauplin 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 PR @lile839 ! Made a few comments to address in order to get things done correctly. Let me know if you have any questions 🤗

Comment on lines +811 to +813
"longcat-video-avatar-1.5": {
prettyLabel: "LongCat-Video-Avatar-1.5",
repoName: "LongCat-Video-Avatar-1.5",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"longcat-video-avatar-1.5": {
prettyLabel: "LongCat-Video-Avatar-1.5",
repoName: "LongCat-Video-Avatar-1.5",
"longcat-video": {
prettyLabel: "LongCat-Video",
repoName: "LongCat-Video",

Only a suggestion but I feel that since the library is named LongCat Video (according to https://github.com/meituan-longcat/LongCat-Video), then the tag should be this instead of the exact repo name itself. This is only a suggestion, if you prefer longcat-video-avatar or even longcat-video-avatar-1.5 then go for it (though I find them too long/specific). The important part is that you had

library_name: longcat-video

in the model card metadata of the model.

Since it's a diffusers model, you can also do

library_name: diffusers
tags:
  - longcat-video

or

# preferred version if you want longcat video download count rule
library_name: longcat-video
tags:
  - diffusers

In both cases, both libraries will be detected but the library_name one will be considered as the "main library" which we will use to count download.

repoName: "LongCat-Video-Avatar-1.5",
repoUrl: "https://github.com/meituan-longcat/LongCat-Video",
filter: false,
countDownloads: `path_extension:"safetensors" OR path:"config.json"`,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Counting both safetensors and config files will lead to double-counting download if someone downloads the full repo. The best in this situation is to only count by config.json, which is the default rule. So my advice here is to remove this line and fallback to the default rule

Suggested change
countDownloads: `path_extension:"safetensors" OR path:"config.json"`,

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.

2 participants