-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Gale mandarin fix #1253
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
Gale mandarin fix #1253
Conversation
|
Still working on it. |
|
@danpovey Ready. |
danpovey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, so this is just changes to the GALE Mandarin recipe... I think the original reason for you to get involved in this was to create an example script for nnet3/chain training with iVectors and pitch features?
But perhaps you've decided to commit these changes first and then do that?
egs/gale_mandarin/s5/run.sh
Outdated
| AUDIO_PATH=/export/corpora5/LDC/LDC2013S08/ | ||
| TEXT_PATH=/export/corpora5/LDC/LDC2013T20/ | ||
|
|
||
| nJobs=64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you please rename these variables to num_jobs and num_jobs_decode ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
|
This PR is for addressing #1163 . I'll created the pitch example using hkust, which is not so heavy to train. |
|
OK, thanks! Merging. |
This PR is based on Yenda's #1207 for addressing #1163