Closed MarcCote closed 9 years ago
A module named modifier
seems awfully undescriptive... direction_modifiers
?
I agree it is very undescriptive but it also contains param_modifier
not only direction_modifier
.
AFAIK, there's no param_modifier
yet, right? If that's the case, we could have a second directory to store the param_modifier
eventually.
This PR is mainly a small refactoring of the classes
Optimizer
andLoss
. In bonus, it adds a new optimizerADAGRAD
and some integration tests for bothSGD
andADAGRAD
optimizers.Closes #34