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.

Sign up for GitHub

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

Jump to bottom

Remove unused option in train_ubm.sh #2592

Closed

david-ryan-snyder opened this issue Aug 5, 2018 · 1 comment

Contributor

david-ryan-snyder commented Aug 5, 2018

Making an issue for this so that I don't lost track of it again.

Dan:

We should remove the nj parameter from train_ubm.sh and make sure none
of the example scripts use it, since it's never used. This is true
for both steps/train_diag_ubm.sh and
steps/online/nnet2/train_diag_ubm.sh (and possibly other scripts
too).

Also, we should reduce the --num-threads default from 32 to 16.

david-ryan-snyder mentioned this issue

[egs,scripts] Remove unused --nj option from some scripts #2679

Merged

Contributor Author

david-ryan-snyder commented Sep 6, 2018

@danpovey, this can also be closed

danpovey closed this as completed

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