-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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] Fix TTS Primer image markup #5192
Conversation
Signed-off-by: Ryan <rlangman@nvidia.com>
@@ -144,7 +144,7 @@ | |||
"While this is the most common structure, there may be fewer or additional steps depending on the use case. For example, some languages do not require G2P and can instead rely on the model to convert raw text/graphemes to spectrogram.\n", | |||
"\n", | |||
"<div>\n", | |||
"<img src=\"https://raw.githubusercontent.com/NVIDIA/NeMo/tts_primer/tutorials/tts/images/tts_pipeline.png\" width=\"500\", height=\"500\"/>\n", | |||
"<img src=\"https://raw.githubusercontent.com/NVIDIA/NeMo/main/tutorials/tts/images/tts_pipeline.png\" width=\"500\", height=\"500\"/>\n", |
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.
These should probably be stable, which means they'll be broken for now. The alternative is to leave it as main
, but you have to update it to stable in the next rc
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.
actually, double check with jocelyn or som
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.
I'd say keep it as main
for now; I'd rather have the images linked properly with a small TODO than have it broken (especially as a primer tutorial).
@@ -144,7 +144,7 @@ | |||
"While this is the most common structure, there may be fewer or additional steps depending on the use case. For example, some languages do not require G2P and can instead rely on the model to convert raw text/graphemes to spectrogram.\n", | |||
"\n", | |||
"<div>\n", | |||
"<img src=\"https://raw.githubusercontent.com/NVIDIA/NeMo/tts_primer/tutorials/tts/images/tts_pipeline.png\" width=\"500\", height=\"500\"/>\n", | |||
"<img src=\"https://raw.githubusercontent.com/NVIDIA/NeMo/main/tutorials/tts/images/tts_pipeline.png\" width=\"500\", height=\"500\"/>\n", |
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.
I'd say keep it as main
for now; I'd rather have the images linked properly with a small TODO than have it broken (especially as a primer tutorial).
Signed-off-by: Ryan <rlangman@nvidia.com>
Signed-off-by: Ryan <rlangman@nvidia.com> Signed-off-by: 1-800-bad-code <shane.carroll@utsa.edu>
Signed-off-by: Ryan <rlangman@nvidia.com> Signed-off-by: Hainan Xu <hainanx@nvidia.com>
Signed-off-by: Ryan <rlangman@nvidia.com> Signed-off-by: Hainan Xu <hainanx@nvidia.com>
Signed-off-by: Ryan rlangman@nvidia.com
What does this PR do ?
Fixes markup issues with github code rendering.
Collection: [TTS]
Changelog
Before your PR is "Ready for review"
Pre checks:
PR Type: