Bonmin: Simplify constr_viol_tol value

mbaudin47 · Sep 4, 2024 · 63a8fa4 · 63a8fa4
1 parent 5e6fda4
commit 63a8fa4
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/lib/src/Base/Optim/Bonmin.cxx b/lib/src/Base/Optim/Bonmin.cxx
@@ -165,10 +165,7 @@ void Bonmin::run()
   for (UnsignedInteger i = 0; i < algos.getSize(); ++ i)
     if (!app.options()->SetIntegerValue("bonmin." + algos[i] + "_log_level", 0))
       throw InvalidArgumentException(HERE) << "Bonmin: Invalid parameter for bonmin." << algos[i] << "_log_level";
-  if (getMaximumConstraintError() > 0.0)
-    app.options()->SetNumericValue("constr_viol_tol", getMaximumConstraintError());
-  else
-    app.options()->SetNumericValue("constr_viol_tol", SpecFunc::MinScalar);
+  app.options()->SetNumericValue("constr_viol_tol", std::max(getMaximumConstraintError(), SpecFunc::MinScalar));
   app.options()->SetNumericValue("bound_relax_factor", 0.0);
   GetOptionsFromResourceMap(app.options());