Skip to content

📝 Added Describe Anything library #1408

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sergiopaniego
Copy link
Member

Adding Describe Anything library as a new option for model cards.

Part of this collection

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

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

Looks good! If it's not too complicated, we could also add a code snippet as in most of the other libraries (we can do it in a separate PR too).

Copy link
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.

Looks good! Let's wait for https://huggingface.co/models?other=describe-anything to be populated before merging this PR though (see comment below)

@@ -230,6 +230,12 @@ export const MODEL_LIBRARIES_UI_ELEMENTS = {
filter: false,
countDownloads: `path:"scin_dataset_precomputed_embeddings.npz" OR path:"saved_model.pb"`,
},
"describe-anything": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you update the model card metadata in https://huggingface.co/nvidia/DAM-3B / https://huggingface.co/nvidia/DAM-3B-Video / https://huggingface.co/nvidia/DAM-3B-Self-Contained to include library_name: describe-anything ? This way they will be listed under https://huggingface.co/models?other=describe-anything and therefore more discoverable. We prefer to have this metadata update done on the Hub before merging this PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure!
I've created PRs for the three model cards 😄

@sergiopaniego
Copy link
Member Author

Added code snippet!

@@ -237,6 +237,19 @@ output = model.generate(text)
sf.write("simple.mp3", output, 44100)`,
];

export const describe_anything = (model: ModelData): string[] => [
`from huggingface_hub import snapshot_download
Copy link
Member

Choose a reason for hiding this comment

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

maybe just add a quick line about installing the library in a comment if possible.

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.

4 participants