-
Notifications
You must be signed in to change notification settings - Fork 493
Add KittenML code snippet #1683
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, as long as some PRs like this one are merged in the Hub. (I think the library name there should be kittentts
, not KittenTTS).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @pcuenca's comment! (must be kittentts
in PRs + please ping us once at least one model is listed under https://huggingface.co/models?other=kittentts)
Co-authored-by: Lucain <lucain@huggingface.co>
Hey @Deep-unlearning, I went back to this PR. It is ready to be merged but https://huggingface.co/models?other=kittentts is not populated yet. I saw that you opened PRs like this one https://huggingface.co/KittenML/kitten-tts-nano-0.1/discussions/6/files so please let me know once they are merged. Also regarding https://huggingface.co/KittenML/kitten-tts-nano-0.1/discussions/6/files, the library_name should be set to |
Co-authored-by: Lucain <lucain@huggingface.co>
No description provided.