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

RMSprop, AdaDelta and Adam steppers #89

Closed
wants to merge 10 commits into from
Closed

Conversation

kashif
Copy link

@kashif kashif commented Nov 7, 2015

For issue #46

@flukeskywalker
Copy link
Collaborator

Thanks, looks good! Have you tried training with it yet?

@kashif
Copy link
Author

kashif commented Nov 7, 2015

not yet, let me add some tests and the rest of the steppers etc and then will try it with examples...

should i rename the issue with [WIP] ?

@flukeskywalker
Copy link
Collaborator

Cool. There aren't many issues yet, so it's also fine not to label.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling a570b60 on kashif:rmsprop into 08efb72 on IDSIA:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 5ea08e4 on kashif:rmsprop into 5f82da3 on IDSIA:master.

@kashif kashif changed the title Initial RMSprop stepper RMSprop, AdaDelta steppers Nov 23, 2015
@landscape-bot
Copy link

Code Health
Repository health decreased by 0.11% when pulling 5307959 on kashif:rmsprop into 36feec5 on IDSIA:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 5525bc6 on kashif:rmsprop into 36feec5 on IDSIA:master.

@kashif
Copy link
Author

kashif commented Jan 1, 2016

@flukeskywalker should be ready to merge.

@kashif kashif changed the title RMSprop, AdaDelta steppers RMSprop, AdaDelta and Adam steppers Jan 5, 2016
@flukeskywalker
Copy link
Collaborator

@kashif Back from vacations now, will review and merge soon. HNY!

@kashif
Copy link
Author

kashif commented Jan 5, 2016

no worries @flukeskywalker HNY to you too! 🎉

@kashif kashif closed this Nov 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.

3 participants