fixup

lebuller · Jan 16, 2024 · be1eca6 · be1eca6
1 parent 8d9a2df
commit be1eca6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/grid/src/Cabana_Grid_HypreSemiStructuredSolver.hpp b/grid/src/Cabana_Grid_HypreSemiStructuredSolver.hpp
@@ -328,23 +328,16 @@ class HypreSemiStructuredSolver
         error = HYPRE_SStructGraphAssemble( _graph );
         checkHypreError( error );
 
-        // Create the matrix.
+        // Create the matrix. must be done after graph is assembled
         error = HYPRE_SStructMatrixCreate( _comm, _graph, &_A );
         checkHypreError( error );
 
         // Set the SStruct matrix object type
         error = HYPRE_SStructMatrixSetObjectType( _A, object_type );
         checkHypreError( error );
-    }
 
-    /*!
-        \brief Prepare the hypre matrix to have it's values set
-    */
-    void initializeHypreMatrix()
-    {
-        // Initialize the matrix.
-        auto error = HYPRE_SStructMatrixInitialize( _A );
-        checkHypreError( error );
+        // Prepare the matrix for setting values
+        error = HYPRE_SStructMatrixInitialize( _A ) checkHypreError( error );
     }
 
     /*!

diff --git a/grid/src/Cabana_Grid_HypreStructuredSolver.hpp b/grid/src/Cabana_Grid_HypreStructuredSolver.hpp
@@ -209,7 +209,7 @@ class HypreStructuredSolver
             checkHypreError( error );
         }
 
-        // Create the matrix.
+        // Create the matrix object. Must be done after the stencil is setup
         error = HYPRE_StructMatrixCreate( _comm, _grid, _stencil, &_A );
         checkHypreError( error );
         error = HYPRE_StructMatrixSetSymmetric( _A, is_symmetric );