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

Fix convert_nnet2_to_nnet3.py #1774

Merged
merged 1 commit into from
Jul 24, 2017
Merged

Fix convert_nnet2_to_nnet3.py #1774

merged 1 commit into from
Jul 24, 2017

Conversation

entn-at
Copy link
Contributor

@entn-at entn-at commented Jul 24, 2017

Update convert_nnet2_to_nnet3.py to use 'execute_command' instead of 'run_kaldi_command' and correctly parse/convert SumGroupComponents (tested by converting the nnet2 multisplice TDNN model used in the sre10/v2 example to nnet3).

Use 'execute_command' instead of 'run_kaldi_command' and correctly parse/convert SumGroupComponents
@danpovey danpovey merged commit dbdd284 into kaldi-asr:master Jul 24, 2017
@entn-at entn-at deleted the fix-convert-nnet2-to-nnet3-py branch July 24, 2017 19:36
kronos-cm added a commit to kronos-cm/kaldi that referenced this pull request Jul 25, 2017
* 'master' of https://github.com/kaldi-asr/kaldi: (36 commits)
  [scripts] Fix convert_nnet2_to_nnet3.py (kaldi-asr#1774)
  [egs] Add missing make_corpus_subset.sh in babel_multilang example (kaldi-asr#1766)
  [egs] Graphemic lexicon updates / fixes in babel/s5d recipe and hub4_spanish recipe (kaldi-asr#1740)
  [egs] update hkust results (kaldi-asr#1772)
  [egs] Update AMI chain experiments RE dropout, decay-time and proportional-shrink (kaldi-asr#1732)
  [egs] Fixes to the aishell (Mandarin) recipe (kaldi-asr#1770)
  [egs] Add recipe for aishell data (free Mandarin corpus, 170 hours total) (kaldi-asr#1742)
  [src] Change to arpa-reading code to accept blank lines with whitespace (kaldi-asr#1752)
  [scripts] For nnet3 training, add option to disable the model-combination  (kaldi-asr#1757)
  [scripts] minor bugfix to nnet1 alignment script when creating lattices (kaldi-asr#1764)
  [src] Add support for row/column ranges when reading GeneralMatrix (kaldi-asr#1761)
  [src] Change name of option --norm-mean->--norm-means for consistency, thanks: 415198468@qq.com
  [egs] swbd/s5c, added 5 layer (b)lstm recipes (kaldi-asr#1759)
  [scripts] Fix bug in segment_long_utterances.sh (kaldi-asr#1758)
  [src] Fix indexing error in nnet1::Convolutional2DComponent (kaldi-asr#1755)
  [src] Fix usage message of program (thanks:jubang0219@gmail.com)
  [egs] some small updates to scripts (installing beamformit; segmentation example)
  [egs] Small fix to ami/s5b/local/chain/compare_wer_general.sh (kaldi-asr#1751)
  [build] Add configuration check for incompatible g++ compilers when CUDA is enabled. (kaldi-asr#1749)
  [egs] Update Librispeech nnet3 TDNN recipe (old one did not run) (kaldi-asr#1727)
  ...
Skaiste pushed a commit to Skaiste/idlak that referenced this pull request Sep 26, 2018
Use 'execute_command' instead of 'run_kaldi_command' and correctly parse/convert SumGroupComponents
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