Apply clang format

Author: sean-baccas

framework/include/mfem/bcs/MFEMDGBoundaryCondition.h

@@ -30,6 +30,7 @@ class MFEMDGBoundaryCondition : public MFEMBoundaryCondition
   /// Get name of the trial variable (gridfunction) the kernel acts on.
   /// Defaults to the name of the test variable labelling the weak form.
   virtual const std::string & getTrialVariableName() const { return _test_var_name; }
+
 protected:
   /// Name of (the test variable associated with) the weak form that the kernel is applied to.
   int _fe_order;

framework/include/mfem/dgkernels/MFEMDGKernel.h

@@ -40,7 +40,6 @@ class MFEMDGKernel : public MFEMGeneralUserObject, public MFEMBlockRestrictable
   mfem::ConstantCoefficient _zero;
   mfem::real_t _sigma;
   mfem::real_t _kappa;
-
 };
 
 #endif

framework/include/mfem/equation_systems/EquationSystem.h

@@ -162,18 +162,20 @@ class EquationSystem : public mfem::Operator
       const std::string & trial_var_name,
       const std::string & test_var_name,
       std::shared_ptr<FormType> form,
-      NamedFieldsMap<NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGBoundaryCondition>>>> & bc_map,
+      NamedFieldsMap<NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGBoundaryCondition>>>> &
+          bc_map,
       std::optional<mfem::real_t> scale_factor = std::nullopt);
-  
+
   void ApplyDomainDGLFIntegrators(
       const std::string & test_var_name,
       std::shared_ptr<mfem::ParLinearForm> form,
       NamedFieldsMap<NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGKernel>>>> & kernels_map);
-  
+
   void ApplyBoundaryDGLFIntegrators(
       const std::string & test_var_name,
       std::shared_ptr<mfem::ParLinearForm> form,
-      NamedFieldsMap<NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGBoundaryCondition>>>> & bc_map);
+      NamedFieldsMap<NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGBoundaryCondition>>>> &
+          bc_map);
 
   /// Names of all trial variables of kernels and boundary conditions
   /// added to this EquationSystem.
@@ -350,7 +352,8 @@ EquationSystem::ApplyDomainDGBLFIntegrators(
       mfem::BilinearFormIntegrator * integ = kernel->createBFIntegrator();
       if (integ != nullptr)
       {
-        mooseAssert( !kernel->isSubdomainRestricted(), "AddInteriorFaceIntegrator doesn't have an overload for this case " );
+        mooseAssert(!kernel->isSubdomainRestricted(),
+                    "AddInteriorFaceIntegrator doesn't have an overload for this case ");
         form->AddInteriorFaceIntegrator(std::move(integ));
       }
     }
@@ -417,14 +420,14 @@ EquationSystem::ApplyDomainDGLFIntegrators(
       mfem::LinearFormIntegrator * integ = kernel->createLFIntegrator();
       if (integ != nullptr)
       {
-        mooseAssert( !kernel->isSubdomainRestricted(), "AddInteriorFaceIntegrator doesn't have an overload for this case " );
+        mooseAssert(!kernel->isSubdomainRestricted(),
+                    "AddInteriorFaceIntegrator doesn't have an overload for this case ");
         form->AddInteriorFaceIntegrator(std::move(integ));
       }
     }
   }
 }
 
-
 } // namespace Moose::MFEM
 
 #endif

framework/include/mfem/problem/MFEMProblem.h

@@ -119,8 +119,8 @@ class MFEMProblem : public ExternalProblem
    * Same as addKernel, but this time for DGKernels
    */
   void addDGKernel(const std::string & kernel_name,
-                 const std::string & name,
-                 InputParameters & parameters) override;
+                   const std::string & name,
+                   InputParameters & parameters) override;
 
   void addDGBoundaryCondition(const std::string & kernel_name,
                               const std::string & name,

framework/src/mfem/actions/AddMFEMDGBoundaryConditions.C

@@ -17,8 +17,7 @@ InputParameters
 AddMFEMDGBoundaryConditions::validParams()
 {
   InputParameters params = MooseObjectAction::validParams();
-  params.addClassDescription(
-      "Add in DG Kernel on the boundary");
+  params.addClassDescription("Add in DG Kernel on the boundary");
   return params;
 }
 

framework/src/mfem/bcs/MFEMDGBoundaryCondition.C

@@ -19,16 +19,24 @@ MFEMDGBoundaryCondition::validParams()
   params.addParam<VariableName>("variable",
                                 "Variable labelling the weak form this kernel is added to");
   params.addParam<mfem::real_t>("sigma", -1.0, "One of the DG penalty params. Typically +/- 1.0");
-  params.addParam<mfem::real_t>("kappa", "One of the DG penalty params. Should be positive. Will default to (order+1)^2");
+  params.addParam<mfem::real_t>(
+      "kappa", "One of the DG penalty params. Should be positive. Will default to (order+1)^2");
   return params;
 }
 
 MFEMDGBoundaryCondition::MFEMDGBoundaryCondition(const InputParameters & parameters)
   : MFEMBoundaryCondition(parameters),
-  _fe_order(getMFEMProblem().getProblemData().gridfunctions.Get(_test_var_name)->ParFESpace()->FEColl()->GetOrder())
-  , _one(1.0), _zero(0.0),
-  _sigma(getParam<mfem::real_t>("sigma")),
-  _kappa( (isParamSetByUser("kappa")) ? getParam<mfem::real_t>("kappa") : (_fe_order+1)*(_fe_order+1) )
+    _fe_order(getMFEMProblem()
+                  .getProblemData()
+                  .gridfunctions.Get(_test_var_name)
+                  ->ParFESpace()
+                  ->FEColl()
+                  ->GetOrder()),
+    _one(1.0),
+    _zero(0.0),
+    _sigma(getParam<mfem::real_t>("sigma")),
+    _kappa((isParamSetByUser("kappa")) ? getParam<mfem::real_t>("kappa")
+                                       : (_fe_order + 1) * (_fe_order + 1))
 {
 }
 

framework/src/mfem/dgkernels/MFEMDGKernel.C

@@ -22,19 +22,28 @@ MFEMDGKernel::validParams()
   params.addParam<VariableName>("variable",
                                 "Variable labelling the weak form this kernel is added to");
   params.addParam<mfem::real_t>("sigma", -1.0, "One of the DG penalty params. Typically +/- 1.0");
-  params.addParam<mfem::real_t>("kappa", "One of the DG penalty params. Should be positive. Will default to (order+1)^2");
+  params.addParam<mfem::real_t>(
+      "kappa", "One of the DG penalty params. Should be positive. Will default to (order+1)^2");
   return params;
 }
 
 MFEMDGKernel::MFEMDGKernel(const InputParameters & parameters)
-: MFEMGeneralUserObject(parameters),
-  MFEMBlockRestrictable(parameters,
+  : MFEMGeneralUserObject(parameters),
+    MFEMBlockRestrictable(parameters,
                           getMFEMProblem().getMFEMVariableMesh(getParam<VariableName>("variable"))),
-  _test_var_name(getParam<VariableName>("variable")),
-  _fe_order(getMFEMProblem().getProblemData().gridfunctions.Get(_test_var_name)->ParFESpace()->FEColl()->GetOrder())
-  , _one(1.0), _zero(0.0),
-  _sigma(getParam<mfem::real_t>("sigma")),
-  _kappa( (isParamSetByUser("kappa")) ? getParam<mfem::real_t>("kappa") : (_fe_order+1)*(_fe_order+1) )
-{}
+    _test_var_name(getParam<VariableName>("variable")),
+    _fe_order(getMFEMProblem()
+                  .getProblemData()
+                  .gridfunctions.Get(_test_var_name)
+                  ->ParFESpace()
+                  ->FEColl()
+                  ->GetOrder()),
+    _one(1.0),
+    _zero(0.0),
+    _sigma(getParam<mfem::real_t>("sigma")),
+    _kappa((isParamSetByUser("kappa")) ? getParam<mfem::real_t>("kappa")
+                                       : (_fe_order + 1) * (_fe_order + 1))
+{
+}
 
 #endif

framework/src/mfem/equation_systems/EquationSystem.C

@@ -96,7 +96,7 @@ EquationSystem::AddDGKernel(std::shared_ptr<MFEMDGKernel> kernel)
   {
     auto kernel_field_map =
         std::make_shared<Moose::MFEM::NamedFieldsMap<std::vector<std::shared_ptr<MFEMDGKernel>>>>();
-    _dg_kernels_map.Register(test_var_name, std::move(kernel_field_map));    
+    _dg_kernels_map.Register(test_var_name, std::move(kernel_field_map));
   }
   // Register new kernels map if not present for the test/trial variable
   // pair
@@ -105,7 +105,7 @@ EquationSystem::AddDGKernel(std::shared_ptr<MFEMDGKernel> kernel)
     auto kernels = std::make_shared<std::vector<std::shared_ptr<MFEMDGKernel>>>();
     _dg_kernels_map.Get(test_var_name)->Register(trial_var_name, std::move(kernels));
   }
-  _dg_kernels_map.GetRef(test_var_name).Get(trial_var_name)->push_back(std::move(kernel));  
+  _dg_kernels_map.GetRef(test_var_name).Get(trial_var_name)->push_back(std::move(kernel));
 }
 
 void
@@ -128,10 +128,9 @@ EquationSystem::AddDGBC(std::shared_ptr<MFEMDGBoundaryCondition> bc)
     auto bcs = std::make_shared<std::vector<std::shared_ptr<MFEMDGBoundaryCondition>>>();
     _dg_bc_map.Get(test_var_name)->Register(trial_var_name, std::move(bcs));
   }
-  _dg_bc_map.GetRef(test_var_name).Get(trial_var_name)->push_back(std::move(bc));  
+  _dg_bc_map.GetRef(test_var_name).Get(trial_var_name)->push_back(std::move(bc));
 }
 
-
 void
 EquationSystem::AddIntegratedBC(std::shared_ptr<MFEMIntegratedBC> bc)
 {
@@ -421,7 +420,7 @@ EquationSystem::BuildLinearForms()
     auto lf = _lfs.GetShared(test_var_name);
     ApplyDomainLFIntegrators(test_var_name, lf, _kernels_map);
     ApplyBoundaryLFIntegrators(test_var_name, lf, _integrated_bc_map);
-    
+
     // same with the dg stuff
     ApplyDomainDGLFIntegrators(test_var_name, lf, _dg_kernels_map);
     ApplyBoundaryDGLFIntegrators(test_var_name, lf, _dg_bc_map);
@@ -454,9 +453,9 @@ EquationSystem::BuildBilinearForms()
 
     // and the dg stuff too
     ApplyDomainDGBLFIntegrators<mfem::ParBilinearForm>(
-      test_var_name, test_var_name, blf, _dg_kernels_map);
+        test_var_name, test_var_name, blf, _dg_kernels_map);
     ApplyBoundaryDGBLFIntegrators<mfem::ParBilinearForm>(
-      test_var_name, test_var_name, blf, _dg_bc_map);
+        test_var_name, test_var_name, blf, _dg_bc_map);
     // Assemble
     blf->Assemble();
   }

framework/src/mfem/problem/MFEMProblem.C

@@ -321,7 +321,7 @@ MFEMProblem::addDGKernel(const std::string & kernel_name,
   FEProblemBase::addUserObject(kernel_name, name, parameters);
   const UserObject * kernel_uo = &(getUserObjectBase(name));
 
-  if (dynamic_cast<const MFEMDGKernel*>(kernel_uo)!=nullptr)
+  if (dynamic_cast<const MFEMDGKernel *>(kernel_uo) != nullptr)
   {
     auto object_ptr = getUserObject<MFEMDGKernel>(name).getSharedPtr();
     auto kernel = std::dynamic_pointer_cast<MFEMDGKernel>(object_ptr);
@@ -352,7 +352,7 @@ MFEMProblem::addDGBoundaryCondition(const std::string & bc_name,
   FEProblemBase::addUserObject(bc_name, name, parameters);
   const UserObject * bc_uo = &(getUserObjectBase(name));
 
-  if (dynamic_cast<const MFEMDGBoundaryCondition*>(bc_uo)!=nullptr)
+  if (dynamic_cast<const MFEMDGBoundaryCondition *>(bc_uo) != nullptr)
   {
     auto object_ptr = getUserObject<MFEMDGBoundaryCondition>(name).getSharedPtr();
     auto kernel = std::dynamic_pointer_cast<MFEMDGBoundaryCondition>(object_ptr);
@@ -371,7 +371,6 @@ MFEMProblem::addDGBoundaryCondition(const std::string & bc_name,
   else
   {
     mooseError("Unsupported bc of type '", bc_name, "' and name '", name, "' detected.");
-
   }
 }
 

test/tests/mfem/kernels/tests

@@ -63,7 +63,6 @@
     compute_devices = 'cpu'
     max_parallel = 1 # schemadiff with multiple ranks
     recover = false
-    expect_out = 'Parallel Type:.*distributed.*Mesh Dimension.*3.*Spatial Dimension.*3.*Elems.*2476.*Num Subdomains.*1.*Solver.*MFEMHypreGMRES'
   []
   [MFEMGradDiv]
     type = XMLDiff