[mlpack-git] [mlpack] Weird Condition in SGD (#461)

Arda Aytekin 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 )`][1] or [`bool std::isinf(double arg)`][2], 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!

[1]: http://en.cppreference.com/w/cpp/numeric/math/isnan
[2]: http://en.cppreference.com/w/cpp/numeric/math/isinf

---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/issues/461#issuecomment-149155464
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20151019/56bfad58/attachment.html>


More information about the mlpack-git mailing list