Refactor measurement and printing for future features

.clang-tidy

@@ -19,6 +19,7 @@ Checks: >
     -bugprone-easily-swappable-parameters,
     -bugprone-reserved-identifier,
     readability-*,
+    -readability-math-missing-parentheses,
     -readability-convert-member-functions-to-static,
     -readability-else-after-return,
     -readability-function-cognitive-complexity,
@@ -29,6 +30,7 @@ Checks: >
     -readability-named-parameter,
     -readability-redundant-*,
     -readability-string-compare,
+    -readability-uppercase-literal-suffix,
     performance-*,
     mpi-*,
     openmp-*'

CHANGELOG.md

@@ -57,6 +57,20 @@ The format of this changelog is based on
     lumped port discovery to fail by reducing default from `1e-3` to `1e-12`.
   - Fixed bug in Nastran mesh reader where carriage returns (`\r`) in the mesh file could
     cause a failure to read the mesh.
+  - Changed post-processing, so that appropriate measurements are always written to disk as CSV
+    files. This is a breaking change, since users are now required to specify valid a output
+    folder in `config["Problem"]["Output"]`. Previously an empty string `""` would suppress file
+    printing. ParaView printing is still controlled and suppressed by the `"Save"` or `"SaveStep"`
+    options in the `config["Solver"][...]`.
+  - Changed measurement and printing during post-processing substantially. This change is not
+    user-facing, but will enable future multi-excitation support. All measurements are now
+    performed as part of the `PostOperator` class, which is templated on solver type. The public
+    interface of the `PostOperator` has been simplified: measurements can no longer be called
+    individually, but are grouped in a `MeasurePrintAll` function. The actual printing of
+    measurements has moved out of individual solvers and is orchestrated by the `PostOperatorCSV`
+    class. Other helper classes include a small `Table` class (for data storage and formatting)
+    and a light wrapper `TableWithCSVFile` (with file interaction).
+  - Changed unit conversion interface: this was moved out of `IOData` into a separate `Units` class.
 
 ## [0.13.0] - 2024-05-20
 

CMakeLists.txt

@@ -8,6 +8,8 @@
 # CMake 3.21 was released in Jul. 2021 (required for HIP support)
 cmake_minimum_required(VERSION 3.21)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 # Prohibit in-source builds
 if(CMAKE_SOURCE_DIR STREQUAL CMAKE_BINARY_DIR)
   message(FATAL_ERROR "In-source builds are prohibited")

cmake/ExternalMFEM.cmake

@@ -76,6 +76,9 @@ if(CMAKE_BUILD_TYPE MATCHES "Debug|debug|DEBUG")
   endif()
 endif()
 
+# Replace mfem abort calls with exceptions for testing, default off
+set(PALACE_MFEM_USE_EXCEPTIONS NO)
+
 set(MFEM_OPTIONS ${PALACE_SUPERBUILD_DEFAULT_ARGS})
 list(APPEND MFEM_OPTIONS
   "-DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER}"
@@ -91,6 +94,7 @@ list(APPEND MFEM_OPTIONS
   "-DMFEM_USE_METIS_5=YES"
   "-DMFEM_USE_CEED=NO"
   "-DMFEM_USE_SUNDIALS=${PALACE_WITH_SUNDIALS}"
+  "-DMFEM_USE_EXCEPTIONS=${PALACE_MFEM_USE_EXCEPTIONS}"
 )
 if(PALACE_WITH_STRUMPACK OR PALACE_WITH_MUMPS)
   list(APPEND MFEM_OPTIONS

palace/CMakeLists.txt

@@ -8,6 +8,8 @@
 # CMake 3.21 was released in Jul. 2021 (required for HIP support)
 cmake_minimum_required(VERSION 3.21)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
 # Prohibit in-source builds
 if(CMAKE_SOURCE_DIR STREQUAL CMAKE_BINARY_DIR)
   message(FATAL_ERROR "In-source builds are prohibited")