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

Fisher GAN + Recurrent Highway Network + TTUR #3

Merged
merged 12 commits into from Jul 9, 2017
Merged

Fisher GAN + Recurrent Highway Network + TTUR #3

merged 12 commits into from Jul 9, 2017

Conversation

NickShahML
Copy link
Contributor

This PR consists of the following changes:

  1. Fisher GAN
  2. Recurrent highway network with multiplicative integration and normalizing tanh
  3. Two Time Scale ADAM (this has been set as default)

I have kept as much naming conventions the same as much as possible. One big change made is that all the gru names are replaced with rnn as it allows us to swap different type of rnn cells much more easily assuming they have a state size of n.

On my text editor (sublime), there were some weird spacing issues when I first pulled your repo. Hopefully this doesn't conflict with your editor.

In the meantime, I'm considering cramer gan and have made a corresponding branch on my repo.

@amirbar amirbar merged commit 1ac0e09 into amirbar:master Jul 9, 2017
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.

None yet

2 participants