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

TTS model documentation update #5948

Closed
lumpidu opened this issue Feb 7, 2023 · 4 comments
Closed

TTS model documentation update #5948

lumpidu opened this issue Feb 7, 2023 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation stale TTS

Comments

@lumpidu
Copy link

lumpidu commented Feb 7, 2023

TTS model documentation update

A clear and concise description of what the bug is.

The Readme's for the currently supported TTS models are outdated. E.g. there is no hint on that now VITS is supported, nor how SSL-Fastpitch is to be used and how it improves/deviates from the normal Fastpitch model.

Please list minimal steps or code snippet for us to be able to reproduce the bug

Navigate to the TTS description: https://docs.nvidia.com/deeplearning/nemo/user-guide/docs/en/main/tts/intro.html. Search for VITS docuementation or SSL Fastpitch.

Expected behavior

Thorough documentation of the new language models or how to train and if possible current status and performance metrics.

@lumpidu lumpidu added the bug Something isn't working label Feb 7, 2023
@XuesongYang XuesongYang added documentation Improvements or additions to documentation TTS and removed bug Something isn't working labels Feb 10, 2023
@XuesongYang
Copy link
Collaborator

could you please have a look? Thanks.
@redoctopus for SSL;
@treacker for VITS.

@redoctopus
Copy link
Collaborator

Both of these are still under development as @experimental models, so we don't guarantee that they won't change (and break backwards compatibility) in the short term.

We could add a section in the documentation regarding current experimental models/models under development, though that may also fall behind main occasionally. @XuesongYang thoughts?

@github-actions
Copy link
Contributor

This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the stale label Mar 13, 2023
@XuesongYang
Copy link
Collaborator

we added some docs relevant to the above models though they are still experimental. Closing this issue for now, but please feel free to re-open it anytime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation stale TTS
Projects
None yet
Development

No branches or pull requests

4 participants