-
Notifications
You must be signed in to change notification settings - Fork 85
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
SGD optimizer stub #139
SGD optimizer stub #139
Conversation
This now works. There's an API change to the Once an The optimizer step for |
Thanks for bringing up the API change regarding the network methods. Making the optimizer argument optional and setting up SGD as default sounds like a good idea. Regarding the naming, I think |
First attempt at defining the concrete optimizer procedure as a method of SGD optimizer type.
Currently defining the
minimize
subroutine aselemental
to allow a scalar/array/rank-agnostic interface. It's possible that this won't work for all cases if we discover new requirements but let's try this for the time being.