Merges #16238, #16242, and #16343 to main by blisc · Pull Request #16379 · NVIDIA-NeMo/Speech · GitHub
Skip to content

Merges #16238, #16242, and #16343 to main - #16379

Open
blisc wants to merge 4 commits into
mainfrom
tts-multilingual-eval
Open

blisc wants to merge 4 commits into
mainfrom
tts-multilingual-eval

Conversation

@blisc

@blisc blisc commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Evaluation manifests can contain language tags that differ from the corrected tags in the evalset configuration. Add an opt-in override to use the dataset or command-line language while preserving per-record manifest precedence by default.

Extend comparison reports to recognize multilingual QA datasets and handle unavailable optional similarity metrics and SFTP directory symlinks.

Collection: TTS

Changelog

  • Thread --ignore_manifest_language through inference dataset creation.
  • Register multilingual QA benchmarks and their language codes.
  • Skip unavailable optional metrics, including all-NaN samples, in statistical tests and plots.
  • Follow SFTP symlinks that point to directories and skip dangling links.
  • Add 13 language-precedence cases and document the override.

Usage

Pass --ignore_manifest_language to examples/tts/magpietts_inference.py when the evalset or command-line language should take precedence. Omitting the flag retains the existing behavior for older models.

Additional information

Brings #16238, #16242, and #16343 from the 2608_tts_training branch to main. Formatting is folded into this bundle.

paarthneekhara and others added 4 commits October 14, 2026 11:06
* add an option while running eval to ignore the per record language in the manifest

Signed-off-by: paarthneekhara <paarth.n@gmail.com>

* ignore manifest language argument

Signed-off-by: paarthneekhara <paarth.n@gmail.com>

* restore context duration sampling during evaluation

Use the configured duration range consistently instead of bypassing the minimum for test datasets.

---------

Signed-off-by: paarthneekhara <paarth.n@gmail.com>
(cherry picked from commit 3fd8beb)
Signed-off-by: Jason <jasoli@nvidia.com>
* Add multilingual QA report support

* Make context SSIM distribution optional

* Check optional metric availability explicitly

* Treat all-NaN optional metrics as unavailable

(cherry picked from commit 81bedf3)
Signed-off-by: Jason <jasoli@nvidia.com>
(cherry picked from commit 1ad45ed)
Signed-off-by: Jason <jasoli@nvidia.com>
Signed-off-by: Jason <jasoli@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown

This branch has not been deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants