Skip to content

Mixture model of frame posteriors#2534

Merged
danpovey merged 11 commits intokaldi-asr:masterfrom
teinhonglo:master
Jul 7, 2018
Merged

Mixture model of frame posteriors#2534
danpovey merged 11 commits intokaldi-asr:masterfrom
teinhonglo:master

Conversation

@teinhonglo
Copy link
Copy Markdown
Contributor

The script which combines the outputs of two (or more) different neural networks and then
use the combined posterior probabilities for decoding.

In our experiments, Improvement in WER could be 0.5 ~ 1.5.

@@ -0,0 +1,235 @@
#!/bin/bash

# Copyright 2018 Tien-Hong Lo
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!
I think this should be called 'decode_score_fusion.sh'.
And can you please do more checking on the inputs? You should at least check that they have the same frame-subsampling-factor and the same tree.

@teinhonglo
Copy link
Copy Markdown
Contributor Author

Dear Povey,

Thanks for your advice.
I have already modify my code based on your advice.

  1. rename "score_fusion.sh" to "decode_score_fusion.sh".
  2. checking that they have the same frame-subsampling-factor and tree.

Best Regards,
Tien-Hong


model=$srcdir/$iter.mdl
if [ ! -f $srcdir/$iter.mdl ]; then
echo "$0: ERROR: no such file $srcdir/$iter.raw. Trying $srcdir/$iter.mdl exit" && exit 1;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please say Error, not ERROR.

@teinhonglo
Copy link
Copy Markdown
Contributor Author

Is there anything I could or should do?

@danpovey danpovey merged commit 295877c into kaldi-asr:master Jul 7, 2018
@danpovey
Copy link
Copy Markdown
Contributor

danpovey commented Jul 7, 2018 via email

dpriver pushed a commit to dpriver/kaldi that referenced this pull request Sep 13, 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.

2 participants