fluids: Cleanup unused struct members

Many of these are set, but never actually used.

Drop Coordinate Basis from IJacobian (without breaking IDL) (#1380)

* compute IDL coefficient in IFunction that has access to coordinates, append this to jac_data so that IJacobian can avoid pulling

Drop Coordinate Basis from IJacobian (without breaking IDL) (#1380)

* compute IDL coefficient in IFunction that has access to coordinates, append this to jac_data so that IJacobian can avoid pulling coordinates through basis to apply IDL

Co-authored-by: James Wright <james@jameswright.xyz>

show more ...

boundary integrals get coordinates back as some tests use BI that needs them

Potential energy removed from all state helpers so no more x (or dx) dependence for them. Interior integrals still need x for IDL but boundary elements don't so removed. This touches lots of qfuncti

Potential energy removed from all state helpers so no more x (or dx) dependence for them. Interior integrals still need x for IDL but boundary elements don't so removed. This touches lots of qfunctions that make state calls and a few problems as well

show more ...

boundary integrals get coordinates back as some tests use BI that needs them

Potential energy removed from all state helpers so no more x (or dx) dependence for them. Interior integrals still need x for IDL but boundary elements don't so removed. This touches lots of qfuncti

Potential energy removed from all state helpers so no more x (or dx) dependence for them. Interior integrals still need x for IDL but boundary elements don't so removed. This touches lots of qfunctions that make state calls and a few problems as well

show more ...

Merge pull request #1270 from CEED/jeremy/constantinople

Rename CEED_BASIS_COLLOCATED to CEED_BASIS_NONE

interface - rename CEED_BASIS_COLLOCATED to CEED_BASIS_NONE

Merge pull request #1309 from CEED/jrwrigh/elem_restr_refactor

fluids: Add DMPlexCeedElemRestriction*Create

fluids: Add DMPlexCeedElemRestriction*Create

Replaces CreateRestrictionFromPlex and GetRestrictionForDomain

Merge pull request #1259 from CEED/jrwrigh/fluids-cleanup

fluids: Misc cleanup

fluids: Correct PetscFunction{Return,BeginUser} placement

- Respect the PETSc standard for style (https://petsc.org/release/developers/style/#coding-conventions-and-style)

fluids: Make function names in consistent format

- Mainly remove `_` and acronymns with more than 2 letters should
respect CamelCase (ie. `StgFunction` not `STGFunction`)

Merge pull request #1308 from CEED/jrwrigh/setup_fem

Fluids: Add DMSetupByOrder helper functions

fluids: Add DMSetupByOrder_* helper function

Merge pull request #1303 from CEED/jrwrigh/fluids-setupqfs

fluids: Refactor setupgeo*, calculate dXdx directly for strong BC

fluids: Calculate dXdx for strong BCs directly

ie. don't use q_data_sur

Merge pull request #1275 from CEED/jrwrigh/fluidsCeedCall

fluids: Add PetscCallCeed to capture ceed errors with Petsc

fluids: Add PetscCallCeed

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)

Merge pull request #1234 from CEED/jrwrigh/fluids_64int

fluids: Compatability for PETSc w/ 64bit ints

fluids: Compatability for PETSc w/ 64bit ints

Merge pull request #1237 from CEED/jrwrigh/petsc-success

examples: Use PETSC_SUCCESS in PetscFunctionReturn