benanne / morb

Modular Restricted Boltzmann Machine (RBM) implementation using Theano
GNU General Public License v3.0
172 stars 41 forks source link

Monitors should be plain old Python functions #1

Closed benanne closed 12 years ago

benanne commented 12 years ago

The Monitor baseclass doesn't do anything useful - monitors should just be Python functions that potentially take a StatsCollector and a Units instance as arguments. Should it turn out to be necessary to make them 'richer', then it's better to use annotations, which is more in line with Samplers and ActivationFunctions.