[mlpack-git] [mlpack/mlpack] Attempt to fix failing build for old Armadillo versions. (310e72d)
notifications at github.com
Tue Sep 20 09:51:13 EDT 2016
I'm not sure this is necessary, due to the reasons listed below. In fact, I'd argue it's actually counterproductive. Using + and % instead of || and && makes the code harder to interpret and less efficient.
Reasoning is as follows. MLPACK requires a C++11 compiler. The lowest version of gcc with sufficient C++11 support is gcc 4.8, which is in both Ubuntu 14.04 and RHEL 7. Ubuntu 14.04 has Armadillo 4.200, while RHEL 7 uses EPEL 7, which currently has Armadillo 4.320 (... nudge, nudge).
Support for && and || operators is present in Armadillo 4.200.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the mlpack-git