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

Marcus Edel notifications at github.com
Thu Feb 11 13:12:54 EST 2016


Okay, great so, I think we should merge this code first and then head over to #516. We have to slightly modify #516, I can do that if no one likes. 

> About c++14 feature, could we postpone the use of them? Because there are a lot of c++14 features do not support by vc2013. Although vc2015 out, but cuda do not support vc2015 yet.

The test should build using a C++11 compiler because of the conditional-compilation directives Joseph used. But I agree, we should avoid C++14 features for the moment, we can easily rewrite the test. I can do that once the code is merged.

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


More information about the mlpack-git mailing list