Skip to content

Add dropout to Xconfig#1589

Merged
danpovey merged 1 commit intokaldi-asr:kaldi_52from
hhadian:cifar
Apr 28, 2017
Merged

Add dropout to Xconfig#1589
danpovey merged 1 commit intokaldi-asr:kaldi_52from
hhadian:cifar

Conversation

@hhadian
Copy link
Contributor

@hhadian hhadian commented Apr 28, 2017

No description provided.

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

It gives 78.5% final accuracy on valid set (w/o data augmentation) which is 4% more than similar Keras setup.
I have not used any normalization either.

@danpovey
Copy link
Contributor

Can you rebase this against kaldi_52? Github is showing some changes that I think I already merged.

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

Rebased it

@danpovey danpovey merged commit 932c88b into kaldi-asr:kaldi_52 Apr 28, 2017
@danpovey
Copy link
Contributor

I merged this, but I was surprised that you didn't combine dropout with batchnorm, e.g. conv-relu-batchnorm-dropout-layer.

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

I will do that. First I wanted to try the similar setup as Keras (which does not use batchnorm).

@danpovey
Copy link
Contributor

danpovey commented Apr 28, 2017 via email

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

OK.

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

BTW, I also tried 1c setup w/o dropout, and the final accuracy was 68% and there was significantly more overfitting.

@danpovey
Copy link
Contributor

danpovey commented Apr 28, 2017 via email

@hhadian
Copy link
Contributor Author

hhadian commented Apr 28, 2017

That's interesting. It is also interesting that 1c was very fast to train too (even though it has similar setup to Keras with a rather big fully connected layer 8192x512). I wonder why Keras takes so long to train.

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