Merge pull request #413 from fabinsch/topic/inf-to-max

replace `std::numeric_limits<T>::infinity()` by `max()`

Author: jorisv

CHANGELOG.md

@@ -9,6 +9,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 ### Changed
 - Change the default branch to `devel` ([#395](https://github.com/Simple-Robotics/proxsuite/pull/395))
 - Change `dual_feasibility` test threshold in `sparse_maros_meszaros` unit test ([#403](https://github.com/Simple-Robotics/proxsuite/pull/403))
+- replace `std::numeric_limits<T>::infinity()` by `std::numeric_limits<T>::max()` ([#413](https://github.com/Simple-Robotics/proxsuite/pull/413))
 
 ### Fixed
 - Use the right table to store `configure-args` cmeel argument ([#403](https://github.com/Simple-Robotics/proxsuite/pull/403))

include/proxsuite/proxqp/dense/linesearch.hpp

@@ -418,7 +418,7 @@ primal_dual_ls(const Model<T>& qpmodel,
     return;
   }
   ////////// STEP 2 ///////////
-  auto infty = std::numeric_limits<T>::infinity();
+  auto infty = std::numeric_limits<T>::max();
 
   T last_neg_grad = 0;
   T alpha_last_neg = 0;

include/proxsuite/proxqp/sparse/solver.hpp

@@ -117,7 +117,7 @@ ldl_iter_solve_noalias(
 
   LDLT_TEMP_VEC_UNINIT(T, err, n_tot, stack);
 
-  T prev_err_norm = std::numeric_limits<T>::infinity();
+  T prev_err_norm = std::numeric_limits<T>::max();
 
   for (isize solve_iter = 0; solve_iter < settings.nb_iterative_refinement;
        ++solve_iter) {
@@ -1209,7 +1209,7 @@ qp_solve(Results<T>& results,
               std::unique(alphas.data(), alphas.data() + alphas_count) -
               alphas.data();
             if (alphas_count > 0) { //&& alphas[0] <= 1
-              auto infty = std::numeric_limits<T>::infinity();
+              auto infty = std::numeric_limits<T>::max();
 
               T last_neg_grad = 0;
               T alpha_last_neg = 0;