Skip to content

Update TTS dataloaders and TTS evaluation to better support text normalization - #16159

Open
Edresson wants to merge 9 commits into
NVIDIA-NeMo:mainfrom
Edresson:Main_TN_fixes
Open

Update TTS dataloaders and TTS evaluation to better support text normalization#16159
Edresson wants to merge 9 commits into
NVIDIA-NeMo:mainfrom
Edresson:Main_TN_fixes

Conversation

@Edresson

@Edresson Edresson commented Aug 28, 2026

Copy link
Copy Markdown
Member

Important

The Update branch button must only be pressed in very rare occassions.
An outdated branch is never blocking the merge of a PR.
Please reach out to the automation team before pressing that button.

What does this PR do ?

Change TTS evaluation to use normalized_text for metrics computation and add a parameter to disable the use of normalized_text on training

Collection: [TTS]

Signed-off-by: Edresson Casanova <edresson1@gmail.com>
Signed-off-by: Edresson Casanova <edresson1@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the TTS label Aug 28, 2026
@Edresson
Edresson marked this pull request as draft August 28, 2026 17:08
Comment thread tests/collections/tts/data/test_magpietts_dataset_lhotse.py Fixed
@Edresson Edresson changed the title Change TTS evaluation to use normalized_text when it is available for metrics computation and add a parameter to disable the use of normalized_text on training Aug 28, 2026
@Edresson Edresson changed the title Change TTS evaluation to use normalized_text for metrics computation and add a parameter to disable the use of normalized_text on training Aug 28, 2026
Signed-off-by: Edresson Casanova <edresson1@gmail.com>
@Edresson
Edresson marked this pull request as ready for review August 28, 2026 18:00
@Edresson

Copy link
Copy Markdown
Member Author

/ok to test 95a4101

@NVIDIA-NeMo NVIDIA-NeMo deleted a comment from copy-pr-bot Bot Aug 28, 2026
@NVIDIA-NeMo NVIDIA-NeMo deleted a comment from copy-pr-bot Bot Aug 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

[🤖]: Hi @Edresson 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

Signed-off-by: Edresson Casanova <edresson1@gmail.com>
Signed-off-by: Edresson Casanova <edresson1@gmail.com>
Signed-off-by: Edresson Casanova <edresson1@gmail.com>
@Edresson

Copy link
Copy Markdown
Member Author

/ok to test 3a83f54

Signed-off-by: Edresson Casanova <edresson1@gmail.com>
@Edresson

Copy link
Copy Markdown
Member Author

/ok to test 11fa906

@github-actions

Copy link
Copy Markdown
Contributor

[🤖]: Hi @Edresson 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

@Edresson

Copy link
Copy Markdown
Member Author

/ok to test 4c6a44e

@github-actions

Copy link
Copy Markdown
Contributor

[🤖]: Hi @Edresson 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

@Edresson

Copy link
Copy Markdown
Member Author

/ok to test 0069002

@github-actions

Copy link
Copy Markdown
Contributor

[🤖]: Hi @Edresson 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants