-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[WIP] Add integration tests for ESPnetEZ #5725
base: master
Are you sure you want to change the base?
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
I also encountered the following error when I tried to run the ENF-TSE task.
I noticed that changing the training configuration works for this test, but I think this is not the correct way to fix this because there would be no issue if we ran the integration test from # before
chunk_excluded_key_prefixes:
- "enroll_ref"
# after
chunk_excluded_key_prefixes:
- "enroll_ref"
- "speech_ref" # add this
- "speech_mix" # add this |
What?
Adds integration tests for ESPnet EZ
Why?
To ensure that ESPnetEZ supports the same tasks as espnet2.
See also
#5584