test: Add SGS data-driven sequential test

docs(fluids): Fused/sequential data-driven SGS model

fluids: Implement sequential SGS DD eval

Fixing SmartRedis meta data arrays to be of type Int64 (#1501)

* fluids: fixed SmartRedis meta data arrays to be of type Int64

* fluids: style fix

---------

Co-authored-by: Riccardo Balin <

Fixing SmartRedis meta data arrays to be of type Int64 (#1501)

* fluids: fixed SmartRedis meta data arrays to be of type Int64

* fluids: style fix

---------

Co-authored-by: Riccardo Balin <balin@aurora-uan-0010.hostmgmt.cm.aurora.alcf.anl.gov>

show more ...

Merge pull request #1496 from CEED/jrwrigh/ksp_mass

Fluids: Fix mass operator for explicit timestepping, move to KSP for mass operator

fixup! fluids: Use KSP for explicit mass matrix

fluids: Fix mass matrix for explicit time stepping

The local vector should be initialized to zero so that all the essential
nodes (ie. strong boundary condition nodes) are not factored in for the
ma

fluids: Fix mass matrix for explicit time stepping

The local vector should be initialized to zero so that all the essential
nodes (ie. strong boundary condition nodes) are not factored in for the
matrix multiply. The PR associated with this commit should have a
document attached detailing why this is the case.

Previous tests used this incorrect mass matrix and so now fail. The
results for these tests are corrected as well.

show more ...

fluids: Destroy stray Mats

fluids: Use KSP for explicit mass matrix

fluids: Remove convergence test from velocity_gradient_projection

Testing with GDB, the `Mass_N` QFunction is only called when creating
the rowsum preconditioner and isn't called afterwards. This is

fluids: Remove convergence test from velocity_gradient_projection

Testing with GDB, the `Mass_N` QFunction is only called when creating
the rowsum preconditioner and isn't called afterwards. This is true with
or without the convergence test. So I've removed the convergence test as
it's not necessary

show more ...

fluids: Add QFs for external SGS DD evaluation

fluids: Refactor function names

- PascalCase for acronyms longer than 2 letters
- Remove the "Anisotropic" qualifier from function names, as we only
have one data-driven SGS model. If we add an is

fluids: Refactor function names

- PascalCase for acronyms longer than 2 letters
- Remove the "Anisotropic" qualifier from function names, as we only
have one data-driven SGS model. If we add an isotropic model later, we
can give that the "Isotropic" qualifier on it's function names
- Remove the "Model" qualifier from function names. "SgsDD" is
communicative enough (Subgrid-stress inherently means a model)

show more ...

fluids: Change SGS IFunctionApply to use NewtonianIdealGasContext

- It doesn't need anything more than that

fluids: Create generic SGS DD eval interface

- Existing implementation now called "Fused" evaluation

fluids: Include config.mk from top level

Now `SMARTREDIS_DIR` can be added to the config.mk and it passed into
the fluids Makefile

fluids: Add LogEvent for VeloGradProj and DiffFilter

Merge balin/smartsim branch into main with timers for SmartRedis API calls (#1480)


---------

Co-authored-by: James Wright <james@jameswright.xyz>
Co-authored-by: Riccardo Balin <balin@aurora-u

Merge balin/smartsim branch into main with timers for SmartRedis API calls (#1480)


---------

Co-authored-by: James Wright <james@jameswright.xyz>
Co-authored-by: Riccardo Balin <balin@aurora-uan-0012.hostmgmt.cm.aurora.alcf.anl.gov>

show more ...

fluids: Replace if...SETERRQ with PetscCheck

fluids: Add missing OperatorApplyContextDestroy's

Merge pull request #1479 from CEED/jrwrigh/fix_mat_reporting

fluids: Report both amat and pmat types

fluids: Report both amat and pmat types

fluids: Use utility for velocity gradient projection

- Evidentily missed this one when I added
StatePhysicalGradientFromReference awhile back. Or this was on a
longer lived branch that got me

fluids: Use utility for velocity gradient projection

- Evidentily missed this one when I added
StatePhysicalGradientFromReference awhile back. Or this was on a
longer lived branch that got merged out-of-order

show more ...

docs(fluids): Document changes to 2D/3D advection

fluids: Merge 2D and 3D advection setup

Instead of `-problem advection` and `-problem advection2d`, there's just
`-problem advection` and the dimensionality is taken from the dm.

fluids: Generalize advection boundary qfunctions