[pymvpa] tiny suggestion for SOM (10 second change)
Ike Hall
ike.hall at gmail.com
Sun Mar 10 22:47:01 UTC 2013
It would appear that the SimpleSOMMapper class performs the initialization
of it's Kohonen layer in the SimpleSOMMapper._train method. As the
.train() method calls ._pretrain, ._train, and ._posttrain in order, I
submit that it would be preferable to move the initialization phase to
._pretrain. With this change, if one wants to subclass the SimpleSOMMapper
to use some other initialization, they only need to override ._pretrain
rather than _train.
--Ike Hall
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-exppsy-pymvpa/attachments/20130310/b7fcd14e/attachment.html>
More information about the Pkg-ExpPsy-PyMVPA
mailing list