[mlpack-git] [mlpack] Adding Copy Ctor to ANN (#519)

Marcus Edel notifications at github.com
Wed Feb 24 11:12:43 EST 2016


In 69f97e50aeddfb83e, f6c27ed38633f732d1e8d and 43f0f13fe84e117 we removed the optimizer from each layer in favor of a single optimizer in the base class (FFN, CNN, RNN). Not only makes this changes the layer easier, we can also use the existing mlpack optimizer for the neural network code.

Since this pull request works with the "one optimizer for each layer" it didn't completely apply anymore. However, I would like to merge the copy Ctor test, so maybe you can open another pull request?  If not I'll go and extract the changes from the pull request.

Please comment if something doesn't make sense and I'll reopen it.

---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/519#issuecomment-188326916
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160224/01854a42/attachment.html>


More information about the mlpack-git mailing list