Removed nan setting

polyfem · Feb 8, 2024 · 1d5566b · 1d5566b
1 parent ba80533
commit 1d5566b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/polysolve/nonlinear/Problem.hpp b/src/polysolve/nonlinear/Problem.hpp
@@ -13,7 +13,7 @@ namespace polysolve::nonlinear
     class Problem
     {
     public:
-        static const int Dim = Eigen::Dynamic;
+        static constexpr int Dim = Eigen::Dynamic;
         using Scalar = double;
         using TVector = Eigen::Matrix<Scalar, Dim, 1>;
         using TMatrix = Eigen::Matrix<Scalar, Dim, Dim>;

diff --git a/src/polysolve/nonlinear/Solver.cpp b/src/polysolve/nonlinear/Solver.cpp
@@ -286,17 +286,10 @@ namespace polysolve::nonlinear
         update_solver_info(objFunc(x));
         objFunc.post_step(PostStepData(m_current.iterations, solver_info, x, grad));
 
-        // Used for logging
-        // double xDelta = 0, gradNorm = 0;
-
         do
         {
             m_line_search->set_is_final_strategy(m_descent_strategy == m_strategies.size() - 1);
 
-            m_current.xDelta = NaN;
-            m_current.fDelta = NaN;
-            m_current.gradNorm = NaN;
-
             // --- Energy ------------------------------------------------------
 
             double energy;
@@ -390,9 +383,7 @@ namespace polysolve::nonlinear
             if (m_status != Status::Continue)
                 break;
 
-            // ---------------
-            // Variable update
-            // ---------------
+            // --- Variable update ---------------------------------------------
 
             m_logger.trace(
                 "[{}][{}] pre LS iter={:d} f={:g} ‖∇f‖={:g}",