[mlpack-git] [mlpack] Weird Condition in SGD (#461)
notifications at github.com
Mon Oct 19 05:21:29 EDT 2015
Yes, I agree with @stereomatchingkiss . My intention was not to disturb you with the subtleties like this, but to understand what was going on. Yet, if your style guide is not strictly against the use of C++11 or above, you might consider using [`bool std::isnan( double arg )`] or [`bool std::isinf(double arg)`], both from the `#include <cmath>` library. That would make the code easier to read and more robust against compiler-dependent implementation issues. Thanks for the discussion!
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the mlpack-git