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

[Marcus Edel]

Ooh this is a really neat idea, this totally eliminates the function parameter. This would also make the serialization easier, the only drawback I see so far is that any layer that currently holds an optimizer has to implement three more functions.

In #513 @stereomatchingkiss suggested a switch from a reference to pointers to hold the function to be optimized. Primarily, to support serialization for the existing layer. Since @stereomatchingkiss worked on the problem I would like to hear his opinion about this.

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