[mlpack] [mlpack/mlpack] Added copy constructor, move constructor and copy assignment operator. (#826)
Ryan Curtin
notifications at github.com
Mon Dec 12 11:46:42 EST 2016
Hi there,
Thanks for the contribution. Could you please add some tests for these? Take a look at some tests I recently wrote in a different branch: https://github.com/rcurtin/mlpack/blob/bindings/src/mlpack/tests/lsh_test.cpp#L840
Once there are working tests I'll give it a full review.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/826#issuecomment-266482640
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://knife.lugatgt.org/pipermail/mlpack/attachments/20161212/8e2ceb27/attachment.html>
More information about the mlpack
mailing list