Fixed parameter management bugs in RandFGMRES and FGMRES classes.

ORNL · Jun 13, 2024 · 4f83647 · 4f83647
1 parent db597be
commit 4f83647
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/resolve/LinSolverIterativeFGMRES.cpp b/resolve/LinSolverIterativeFGMRES.cpp
@@ -413,7 +413,7 @@ namespace ReSolve
         setRestart(atoi(value.c_str()));
         break;
       case CONV_COND:
-        setRestart(atoi(value.c_str()));
+        setConvCond(atoi(value.c_str()));
         break;
       case FLEXIBLE:
         bool is_flexible = (value == "yes");
@@ -474,6 +474,7 @@ namespace ReSolve
     {
       case FLEXIBLE:
         value = getFlexible();
+        break;
       default:
         out::error() << "Unknown boolean parameter " << id << "\n";
         return 1;
@@ -491,6 +492,9 @@ namespace ReSolve
     case MAXIT:
       std::cout << getMaxit() << "\n";
       break;
+    case RESTART:
+      std::cout << getRestart() << "\n";
+      break;
     default:
       out::error() << "Unknown parameter " << id << "\n";
       return 1;

diff --git a/resolve/LinSolverIterativeRandFGMRES.cpp b/resolve/LinSolverIterativeRandFGMRES.cpp
@@ -519,7 +519,7 @@ namespace ReSolve
         setRestart(atoi(value.c_str()));
         break;
       case CONV_COND:
-        setRestart(atoi(value.c_str()));
+        setConvCond(atoi(value.c_str()));
         break;
       case FLEXIBLE:
         bool is_flexible = (value == "yes");
@@ -580,6 +580,7 @@ namespace ReSolve
     {
       case FLEXIBLE:
         value = getFlexible();
+        break;
       default:
         out::error() << "Unknown boolean parameter " << id << "\n";
         return 1;
@@ -597,6 +598,9 @@ namespace ReSolve
     case MAXIT:
       std::cout << getMaxit() << "\n";
       break;
+    case RESTART:
+      std::cout << getRestart() << "\n";
+      break;
     default:
       out::error() << "Unknown parameter " << id << "\n";
       return 1;