Skip to content
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

Add countDownloads for Oasis #1007

Merged
merged 5 commits into from
Nov 4, 2024
Merged

Add countDownloads for Oasis #1007

merged 5 commits into from
Nov 4, 2024

Conversation

julian-q
Copy link
Contributor

This PR is intended to add download counting support for https://huggingface.co/Etched/oasis-500m. It matches .pt files in order to match both the autoencoder and the diffusion model.
The relevant github repository (used for inferencing the model) is https://github.com/etched-ai/open-oasis.

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.

Thanks @julian-q for the PR :) I've made a few comments to address.
Before getting this merged, there should be at least one model listed on https://huggingface.co/models?other=open-oasis. To do that, you need to update the repo model card and add library_name: open-oasis in the metadata section. This way, the Hub will be able to link your model to your library.

packages/tasks/src/model-libraries.ts Outdated Show resolved Hide resolved
packages/tasks/src/model-libraries.ts Outdated Show resolved Hide resolved
packages/tasks/src/model-libraries.ts Outdated Show resolved Hide resolved
@julian-q
Copy link
Contributor Author

julian-q commented Nov 2, 2024

Fantastic, thank you so much for the fixes @Wauplin . I see the model in https://huggingface.co/models?other=open-oasis! Is everything good to merge now?

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.

All good! Thanks for the quick changes 🤗

@julian-q julian-q requested a review from ngxson as a code owner November 2, 2024 14:57
@Wauplin Wauplin merged commit 56f465e into huggingface:main Nov 4, 2024
1 of 4 checks passed
@coyotte508
Copy link
Member

Hmm maybe we should add a "build" as required step before merging 🤔 (open-oasis was lacking surrounding quotes)

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.

3 participants