Add integration tests for ESPnetEZ#5725
Conversation
- add VITS test
…ord argument 'text_name'`
- `text_name` has default value defined in `CommonPreprocessor_multi` class, so we can simply delete this line.
- If we have multiple speakers with enh_s2t task, we have `text_spk{n}` as input. These should be defined in the configuration as `text_name`, which cause the above error.
- When config for fine-tune does not contain `preprocessor_conf`, it was replaced with empty dict. - After this fix, only when fine-tune config contains `preprocessor_conf` the values will be replaced with the new values.
- No finetune test for hubert, since there is no inference class found.
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5725 +/- ##
===========================================
- Coverage 71.58% 35.62% -35.96%
===========================================
Files 760 758 -2
Lines 69841 69825 -16
===========================================
- Hits 49996 24878 -25118
- Misses 19845 44947 +25102
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
for more information, see https://pre-commit.ci
- This recipe will test MT task.
for more information, see https://pre-commit.ci
|
Hi, when I try to run the integration test for the |
|
@kan-bayashi, can you help the issue? |
for more information, see https://pre-commit.ci
- Changed to the original value
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
- The CI issue comes from the config for new transducer task.
- I have changed the config name to support configuration test for new transducer test.
- Config test does not have test for GAN-TTS, and difficult to support. To avoid error in CI, the config was moved to child directory. - `preprosessor` is removed from config, as `enh_asr1` recipe does not have this argument.
- ENH_ASR task does not have configuration for `preprocessor`, but has `use_preprocessor`. - We need to define `text_name` separately for this task, whereas other tasks defines this argument in `preprocessor_conf.text_name`
for more information, see https://pre-commit.ci
|
Thanks! |
Add integration tests for ESPnetEZ
What?
Adds integration tests for ESPnet EZ
Why?
To ensure that ESPnetEZ supports the same tasks as espnet2.
See also
#5584