-
Notifications
You must be signed in to change notification settings - Fork 664
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
Revise parameters for Kaldi fbank compatibility test #679
Comments
This is very cool. One only comment is that Callhome is used for diarization, not for ASR. |
Hi @sw005320 Thanks for the comment.
Noted. will reflect it.
Do you mean adding an equivalent implementation of |
Yes, exactly. We found that the pitch feature always improved the performance for several tonal languages (e.g., Chinese), and did not degrade the performance for the other languages. |
I see. I created the issue #686 to track this. |
Sorry, I did not notice it. We did not specifically check the feature extraction compatibility test. But we’ll also use torchaudio features and will have make such a test in the near future. |
…ials Reorganize beginner tutorials
Test parameters for Kaldi fbank were generated using this script, but this does not necessarily generate values in valid range, so we need to revise them.
First, we need to test the parity of default values.
Second, we should cover values found in Kaldi examples. such as
Then we can also add perturbation
--frame-length
and--frame-shift
: integer value around 10 - 50 [ms] if perturbing--preemphasis-coefficient
: less than 1.0, around 0.90 - 0.99--low-freq
: around 20 - 50 (and less than--high-freq
)See also #672
The text was updated successfully, but these errors were encountered: