proxsuite icon indicating copy to clipboard operation
proxsuite copied to clipboard

Is the use of infinity required?

Open AccessViolationEnjoyer opened this issue 9 months ago • 2 comments

When building with -ffast-math there are a lot of warnings emitted due to the use of infinity in various places. Can these be replaced with sufficiently large values?

AccessViolationEnjoyer avatar Apr 20 '25 23:04 AccessViolationEnjoyer

Hi @kermado,

I reviewed our code and identified 3 instances where we're using std::numeric_limits<T>::infinity(). I suggest replacing these with a larger number like std::numeric_limits<T>::max() / 2. After making this adjustment, all unit tests passed successfully with -ffast-math.

fabinsch avatar Apr 22 '25 09:04 fabinsch

@fabinsch Could you fix the issue using std::numeric_limits<T>::max()?

jcarpent avatar Jul 29 '25 17:07 jcarpent