Adding the following flags in the Cmake file under the enable_coverage section:

* --ignore-errors mismatch --rc geninfo_unexecuted_blocks=1
* --ignore-errors unused
These flags allowed to save a coverage file even in presence of errors of mismatch type.
Fix some parametres setting in the Trilinos GMRES solver: krylov space, max iteration and max restarts.

Author: dcodoni

Code/Source/solver/CMakeLists.txt

@@ -287,9 +287,9 @@ if(ENABLE_COVERAGE)
   # add coverage target
   add_custom_target(coverage
     # gather data
-    COMMAND ${LCOV} --directory . --capture --output-file coverage.info
+    COMMAND ${LCOV} --directory . --capture --output-file coverage.info --ignore-errors mismatch --rc geninfo_unexecuted_blocks=1
     # exclude externals
-    COMMAND ${LCOV} --remove coverage.info -o coverage.info '/usr/*' '/opt/*' "${PROJECT_SOURCE_DIR}/ThirdParty/*" '*/CMakeFiles/run_all_unit_tests.dir/*' '/Library/*' 'v1/*'
+    COMMAND ${LCOV} --remove coverage.info -o coverage.info '/usr/*' '/opt/*' "${PROJECT_SOURCE_DIR}/ThirdParty/*" '/Library/*' 'v1/*' --ignore-errors unused
     # generate report
     COMMAND ${GENHTML} --demangle-cpp -o coverage coverage.info
     WORKING_DIRECTORY ${CMAKE_BINARY_DIR})

Code/Source/solver/trilinos_impl.cpp

@@ -581,11 +581,14 @@ void trilinos_solve_(const Teuchos::RCP<Trilinos> &trilinos_, double *x, const d
                 + Belos::StatusTestDetails  
                 + Belos::IterationDetails;         
 
+  int maxRestarts = int(maxIters / kspace)+1;
+
   belosParams.set("Verbosity", verbosity);
   belosParams.set("Output Frequency", 1);
   belosParams.set("Output Style", 1);  
   belosParams.set("Convergence Tolerance", relTol);
   belosParams.set("Maximum Iterations", maxIters); 
+  belosParams.set("Maximum Restarts", maxRestarts);
 
   std::string solverType;