[egs,scripts,src] Tune image recognition examples; minor small changes. #1682
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Contributor
| Will do it. …On Tue, Jun 13, 2017 at 11:13 AM, Daniel Povey ***@***.***> wrote: @freewym <https://github.com/freewym>, at some point you may want to run the results for the NIPS paper on top of these latest changes. Changing num-minibatches-history to 40.0 (default is 10, I think) gives a small but consistent improvement. And I'm using smaller archives for the svhn setup which helps slightly too. ------------------------------ You can view, comment on, or merge this pull request online at: #1682 Commit Summary - [egs,scripts,src] Tune image recognition examples; minor small changes. File Changes - *A* egs/cifar/v1/local/nnet3/run_cnn_aug_1d.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-0> (134) - *A* egs/cifar/v1/local/nnet3/run_resnet_1c.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-1> (144) - *M* egs/svhn/v1/local/nnet3/run_cnn_aug_1a.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-2> (5) - *A* egs/svhn/v1/local/nnet3/run_cnn_aug_1b.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-3> (120) - *M* egs/svhn/v1/local/nnet3/run_resnet_1b.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-4> (4) - *A* egs/svhn/v1/local/nnet3/run_resnet_1c.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-5> (133) - *M* egs/svhn/v1/run.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-6> (6) - *M* egs/tedlium/s5_r2/local/chain/tuning/run_tdnn_1e.sh <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-7> (6) - *M* egs/wsj/s5/steps/info/nnet3_dir_info.pl <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-8> (4) - *M* src/nnet3/nnet-simple-component.cc <https://github.com/kaldi-asr/kaldi/pull/1682/files#diff-9> (7) Patch Links: - https://github.com/kaldi-asr/kaldi/pull/1682.patch - https://github.com/kaldi-asr/kaldi/pull/1682.diff — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1682>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADWAkkoq3wqaJ1WsbKFxPmDsRfxfHKKPks5sDtFAgaJpZM4N43d1> . -- Yiming Wang Department of Computer Science The Johns Hopkins University 3400 N. Charles St. Baltimore, MD 21218 |
kronos-cm added a commit to kronos-cm/kaldi that referenced this pull request Jun 15, 2017
* 'master' of https://github.com/kaldi-asr/kaldi: (140 commits) [egs] Fix failure in multilingual BABEL recipe (regenerate cmvn.scp) (kaldi-asr#1686) [src,scripts,egs] Backstitch code+scripts, and one experiment, will add more later. (kaldi-asr#1605) [egs] CNN+TDNN+LSTM experiments on AMI (kaldi-asr#1685) [egs,scripts,src] Tune image recognition examples; minor small changes. (kaldi-asr#1682) [src] Fix bug in looped computation (kaldi-asr#1673) [build] when installing sequitur and mmseg, look for lib64 as well (thanks: @akshayc11) (kaldi-asr#1677) [src] fix to gst-plugin/Makefile (remove -lkaldi-thread) (kaldi-asr#1680) [src] Cosmetic fixes to usage messages [egs] Fix to some --proportional-shrink related example scripts (kaldi-asr#1674) [build] Fix small bug in configure [scripts] Fix small bug in utils/gen_topo.pl. [scripts] Add python script to convert nnet2 to nnet3 models (kaldi-asr#1611) [doc] Fix typo (kaldi-asr#1669) [src] nnet3: fix small bug in checking code. Thanks: @Maddin2000. [src] Add #include missing from previous commit [src] Fix bug in online2-nnet3 decoding RE dropout+batch-norm (thanks: Wonkyum Lee) [scripts] make errors getting report non-fatal (thx: Miguel Jette); add comment RE dropout proportion [src,scripts] Use ConstFst or decoding (half the memory; slightly faster). (kaldi-asr#1661) [src] keyword search tools: fix Minimize() call, necessary due to OpenFst upgrade (kaldi-asr#1663) [scripts] do not fail if the ivector extractor belongs to different user (kaldi-asr#1662) ...
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
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
@freewym, at some point you may want to run the results for the NIPS paper on top of these latest changes. Changing num-minibatches-history to 40.0 (default is 10, I think) gives a small but consistent improvement. And I'm using smaller archives for the svhn setup which helps slightly too.