Skip to content
Merged
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions packages/tasks/src/model-libraries.ts
Original file line number Diff line number Diff line change
Expand Up @@ -716,6 +716,12 @@ export const MODEL_LIBRARIES_UI_ELEMENTS = {
repoName: "k2",
repoUrl: "https://github.com/k2-fsa/k2",
},
lyra: {
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.

Will it cover both https://huggingface.co/nvidia/Lyra and https://huggingface.co/nvidia/Lyra-2.0 ? It yes, then the download count rule is not correct. If it's only for the Lyra 2.0 release then should we tag the library as lyra-2.0 instead?

(also fine to keep it as-is if you think it'd best, I don't have the full picture)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that repo has config.json already, I was thinking for upcoming Lyra models

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done also cc @pcuenca

prettyLabel: "Lyra",
repoName: "Lyra",
repoUrl: "https://github.com/nv-tlabs/lyra",
filter: false,
countDownloads: `path:"checkpoints/image_encoder/model.pth"`,
Comment thread
pcuenca marked this conversation as resolved.
lagernvs: {
prettyLabel: "LagerNVS",
repoName: "LagerNVS",
Expand Down
Loading