search

IDSIA / brainstorm

Fast, flexible and fun neural networks.
Other
1.3k stars 152 forks source link

Steppers VS Step #58

Closed Qwlouse closed 9 years ago

Qwlouse commented 9 years ago

There is a bit of confusion regarding the naming of the TrainingSteps.
What is our decision on that?

  1. if we go with ...Step we should rename the file from steppers.py to steps.py and the attributes and arguments within the trainer should be renamed too.
  2. if we go with Stepper we should rename the base class to TrainingStepper and all derived classes should be renamed too (e.g. SgdStepper).
  3. A different name? Optimizer?

I tend a bit towards Stepper because it sounds like something that does a step. But a different name is also fine with me. In any case this is an API breaking change so it should be decided before release.

flukeskywalker commented 9 years ago

I think I prefer Stepper.