remove NT typename from max_step()

TolisChal · Jul 7, 2024 · 259ddc8 · 259ddc8
1 parent 9d5a13f
commit 259ddc8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/include/preprocess/barrier_center_ellipsoid.hpp b/include/preprocess/barrier_center_ellipsoid.hpp
@@ -69,7 +69,7 @@ std::tuple<MT_dense, VT, bool>  barrier_center_ellipsoid_linear_ineq(MT const& A
         d.noalias() = - solve_vec<NT>(llt, H, grad);
         Ad.noalias() = A * d;
         // Compute the step length
-        step = std::min(max_step_multiplier * get_max_step<VT, NT>(Ad, b_Ax), step_iter);
+        step = std::min(max_step_multiplier * get_max_step(Ad, b_Ax), step_iter);
         step_d.noalias() = step*d;
         x_prev = x;
         x += step_d;

diff --git a/include/preprocess/rounding_util_functions.hpp b/include/preprocess/rounding_util_functions.hpp
@@ -31,9 +31,10 @@ struct AssertBarrierFalseType : std::false_type {};
 template <typename T>
 struct AssertFalseType : std::false_type {};
 
-template <typename VT, typename NT>
-NT get_max_step(VT const& Ad, VT const& b_Ax)
+template <typename VT>
+auto get_max_step(VT const& Ad, VT const& b_Ax)
 {
+    using NT = typename VT::Scalar;
     const int m = Ad.size();
     NT max_element = std::numeric_limits<NT>::lowest(), max_element_temp;
     for (int i = 0; i < m; i++) {