Skip to content
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

GNU parallel to run zrc2019 training in multiple cores #68

Merged
merged 3 commits into from
Nov 30, 2018

Conversation

jubenjum
Copy link
Contributor

I made a mod in the run.sh script and include a new aud_gnu_parallel.sh script that uses gnu parallel to make the training in multiple cores. It may interesting to speeding up the training in single computer with multiple available cores.

Juan

@lucasondel
Copy link
Contributor

Hi Juan,

Thanks a lot for your PR ! I think it's a good strategy to use gnu parallel so everyone can run it.

I've tried to run it on some data and spotted 2 problems:

  1. When you split the list of utterances, "split" may break the file in the middle of a line which make the script to failed for some utterances. You can fix it with

$ split ... -n $njobs ... -> $ split ... -n l/$njobs ...

  1. So far the script just run 1 epoch, it would probably be better if the user specify the number of training epochs and the number of jobs.

Lucas

@jubenjum
Copy link
Contributor Author

Hi Lucas,

I fixed the bug in the split function, and I changed the part that does the epochs. I also introduced a shuffle on the input data then each epoch will be a bit different.

Do you think that the decoding part can be parallelized?

Juan

@lucasondel
Copy link
Contributor

Hi Juan,

Ok it works on my side. I'll merge the PR.

Regarding the parallel decoding. Shouldn't be too complicated to do. I can handle it next week.

Lucas

@lucasondel lucasondel mentioned this pull request Nov 30, 2018
@lucasondel lucasondel merged commit 0c98344 into beer-asr:master Nov 30, 2018
@lucasondel
Copy link
Contributor

Parallel decoding is done and available on the master #70 (see steps/aud_decode_gnu_parallel.sh).

Lucas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants