Skip to content

Conversation

@ChunChiehChang
Copy link
Contributor

I haven't updated the new results on the top of the scripts yet.

Chun-Chieh Chang and others added 30 commits July 21, 2017 13:33
merging the files from the IAM_eng branch to use that file structure
…keep consistent with aarora8/IAM_eng branch
Keeping up to date with code written by aarora8
steps/decode.sh --nj $nj --cmd $cmd exp/mono/graph data/test \
exp/mono/decode_test
fi
#if [ $stage -le 5 ]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are these commented-out parts supposed to be commented out?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I intentionally commented out the decoding parts because the results from the first couple of stages (monophone/triphone) are not very good and I didn't think I needed to compute them. I can just pass their alignments on to the later CNN stages of the script.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than comment them out, you can set at the top
decode_gmm=false
and do things like
if [ $stage -le 5 ] && $decode_gmm; then

@@ -0,0 +1,138 @@
#!/usr/bin/env python3
Copy link
Contributor

@aarora8 aarora8 Apr 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A cleaned version of this file './local/remove_test_utterances_from_lob.py' is already part of kaldi/master : kaldi/egs/iam/v1.

@danpovey
Copy link
Contributor

I am thinking of merging this within 24 hours-- I want to be a bit more aggressive about merging these recipes as I think it will help our work move faster.
@hhadian do you have time to review in a bit more detail? If you are busy it's OK, I can merge and we can address things as we find them.

@danpovey
Copy link
Contributor

oh wait, I see this PR is very small. Merging.

@danpovey danpovey merged commit ccd50e2 into kaldi-asr:master Apr 13, 2018
LvHang pushed a commit to LvHang/kaldi that referenced this pull request Apr 14, 2018
Skaiste pushed a commit to Skaiste/idlak that referenced this pull request Sep 26, 2018
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.

4 participants