Skip to content

Conversation

Deep-unlearning
Copy link
Contributor

No description provided.

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, as long as some PRs like this one are merged in the Hub. (I think the library name there should be kittentts, not KittenTTS).

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.

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)

@Wauplin
Copy link
Contributor

Wauplin commented Aug 28, 2025

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 kittentts as defined in this PR, not KittenTTS.

Co-authored-by: Lucain <lucain@huggingface.co>
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