fluids: Calculate dXdx for strong BCs directly

ie. don't use q_data_sur

fluids: Refactor setupgeo* functionality into helpers

- This allows for use of those helpers inside other qfunctions

Merge pull request #1278 from CEED/jrwrigh/blasius-simplex

fluids: Fix STG and Blasius for non-box meshes

fluids: Use enum for mesh transformation flag

fluids: Remove old, unused STG method (non-Ceed)

- Remove the old method for STG that uses DMAddBoundary. Completely
superseded by Ceed implementation

fluids: Specify streamwise element length for STG

- Generally for non-DMPlex box meshes, where we can't query number of
box faces.

fluids: Explicit flag for blasius mesh modification

- So that blasius example can run using external meshes without any
attempt to modify them

Merge pull request #1297 from CEED/jed/fix-fluids-petsc-stage-log

examples/fluids: fix deprecated/dysfunctional PetscLogGetStageLog

examples/fluids: fix deprecated/dysfunctional PetscLogGetStageLog

As a result of Toby Isaac's refactoring of the stage/events system in
PETSc (for maintainability and some new features), we need thi

examples/fluids: fix deprecated/dysfunctional PetscLogGetStageLog

As a result of Toby Isaac's refactoring of the stage/events system in
PETSc (for maintainability and some new features), we need this pattern.

show more ...

Merge pull request #1286 from CEED/jrwrigh/refactor_gravity

fluids: Refactor gravity option

fluids: Refactor gravity option

- Use `-gravity` instead of `-g`
- Enforce that the input should be a length 3 vector
- Update documentation

Misc: Also update documented default value for `mu`.

Merge pull request #1282 from CEED/jrwrigh/fix-dim-options

fluids: Reset dim before reference_velocity option

fix(fluids): Reset dim before reference_velocity option

If `-g` is not given, then
`PetscOptionsRealArray(..., "-g", ...)`
will set `dim` to zero. This then causes
`PetscOptionsScalarArray(..., "-re

fix(fluids): Reset dim before reference_velocity option

If `-g` is not given, then
`PetscOptionsRealArray(..., "-g", ...)`
will set `dim` to zero. This then causes
`PetscOptionsScalarArray(..., "-reference_velocity", ...)`
to not read any values.

show more ...

fluids: Add PetscCallCeed

fluids: Add user argument to problem print functions

PetscFE determines basis for examples/fluids/navierstokes (#1264)

* breaking lots of DL stuff to test PetscFE

* fixed missing header

* fluids: Remove unused `Involute` function

* fluids: Se

PetscFE determines basis for examples/fluids/navierstokes (#1264)

* breaking lots of DL stuff to test PetscFE

* fixed missing header

* fluids: Remove unused `Involute` function

* fluids: Set quadrature points for PetscFE

* fluids: Fix GetRestrictionForDomain, Use CreateBasisFromPlex

* fluids: Remove Q_sur arg from SetupStrongBC_Ceed

* fluids: Remove extra SetClosurePermutationTensor

* fluids: Correct quadrature order to auxillary DMs

- Should factor in q_extra

* fluids: Fix parent Restriction creation for spanstats

* fluids: Correct qdata creation for strong bcs

* fluids: Set tensor closer only for tensor-product elements

Co-authored-by: Jeremy L Thompson <jeremy@jeremylt.org>

* fluids: Make shocktube test less brittle

- Small, insignificant changes to the coordinate locations make a
dramatic difference to the initial condition
- For the test case, there is a node right at the midpoint of the
domain. Adding the epsilon fudge factor makes the if statement more
stable

* fluids: Fix SGS test

- Change the test so that it passes on all hardware. It's a very
sentitive test and the changes to the basis creation changed the
initial conditions *just* enough to make the test not happy. Should
really try and make this more robust at some point...

* fluids: Address misc PR comments

* fluids: Remove quad_mode arg from CreateBasisFromPlex

---------

Co-authored-by: James Wright <james@jameswright.xyz>
Co-authored-by: Jeremy L Thompson <jeremy@jeremylt.org>

show more ...

Merge pull request #1269 from CEED/jeremy/collocated-qpts

Check for compatible num_qpts with CEED_BASIS_COLLOCATED

fluids - build IC qdata on the fly (only used once)

fluids - drop unneeded calls to CeedOperatorSetNumQuadraturePoints

Merge pull request #1266 from CEED/jrwrigh/fluids-mem-leaks

fluids: Fix mem leaks in grid_anisotropy_tensor.c

fluids: Fix mem leaks in grid_anisotropy_tensor.c

Merge pull request #1257 from CEED/jrwrigh/remove_l2_proj_func

fluids: Remove ComputeL2Projection function

fluids: Remove ComputeL2Projection function

- It's superseded by using `NodalProjectionData` + `ApplyCeedOperator*`
+ `KSPSolve`

fluids: Don't show partition stats for serial jobs

- It results in nans which may confuse users.

fluids: Add neighbor statistics, changed output

- Changed "shared" to "ghost"
- Replaced the "nodes = Dofs/5" note with "5-DoF nodes"
- Use MPIU_INT instead of `PetscDataTypeToMPIDataType`