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

AdaDelta v2 #2204

Closed
wants to merge 4 commits into from
Closed

AdaDelta v2 #2204

wants to merge 4 commits into from

Conversation

kevinbache
Copy link
Contributor

This is a minor update to #1122. It brings @mohomran's AdaDelta solver into sync with modern Caffe and fixes the slow tests mentioned in #1122 by reducing the number of iterations for multi-iter tests from 500 down to 4 (as with the Nesterov tests). All tests pass and take a normal amount of time.

@kevinbache
Copy link
Contributor Author

Is there anything else that should be done for this pull request?

@shelhamer
Copy link
Member

Thanks for carrying on #1122 and taking care of tests and examples. Could you squash all the lint commits (58c5b4c, f5485f7, 31a299a, f2b29ec, etc.) ?

@shelhamer shelhamer mentioned this pull request Apr 1, 2015
@kevinbache kevinbache force-pushed the adadelta branch 2 times, most recently from 2d01607 to e5a2972 Compare April 2, 2015 19:59
@kevinbache
Copy link
Contributor Author

No problem. I ended up just squashing all of mine -- seemed to make the most sense. Please let me know if there's anything else I should do!

@shelhamer shelhamer added the ES label Apr 2, 2015
@matthiasplappert
Copy link
Contributor

I'm interested in the Ada Delta solver. Is this change likely going to make it into master anytime soon?

@shelhamer
Copy link
Member

Carried on in #2782 -- sorry for the wait on this.

@shelhamer shelhamer closed this Aug 4, 2015
@shelhamer shelhamer removed the ES label Aug 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants