NS: Print summary similar to BPs

NS: style

NS: Add data structures for different test types

NS: Add warning if CtauS or strong_form is used with density_current problem

NS: Add missing CHKERRQ(ierr) & Destroy viewer & bool -> PetscBool

NS: Check tests error by comparing with reference solutions from binary files

NS: scale energy differently in TESTARCGS

NS: Remove extra PetscViewerDestroy() call in TSMonitor_NS() function

This was causing a crash in parallel (not in serial)

NS: remove line between test runs and add output files

NS: Add VecViewFromOptions at the end of execution

This currently works in serial, but not in parallel

NS: Add multiple test runs

These minimal tests test different configurations: the explicit formulation with default (RKF45), the implicit formulation with alpha method, and the implicit formulation

NS: Add multiple test runs

These minimal tests test different configurations: the explicit formulation with default (RKF45), the implicit formulation with alpha method, and the implicit formulation with alpha method + supg stabilization

show more ...

NS: Default BCs are free-slip on all faces

If no boundary conditions are set at command line, there will be free-slip boundary conditions on velocity components on all faces

The use still has the p

NS: Default BCs are free-slip on all faces

If no boundary conditions are set at command line, there will be free-slip boundary conditions on velocity components on all faces

The use still has the possibility of imposing wall boundary conditions (which sets all three components of velocity to homogeneous Dirichelet condition)

If the user specify both conditions, slip and wall, on the same face sets, the program will error

show more ...

NS: Style and comments

NS: Remove periodicity option

This option is temporarily removed since it is currently broken

NS: Add warning if CtauS is used without stabilization

NS: Patch on BCs for different problem types and update docs

NS: Add warning if stabilization is used with explicit time integrators

NS: Remove unused variables and some code cleanups and clarifications

fluids: fix missing CHKERRQ

Merge pull request #526 from CEED/valeria/petsc-examples-cleaning

PETSc examples: some cleaning

NS solver: remove stray restrictxcoord restriction

NS: Use non_zero_time=false for problem type avection

Since exact solution and error calculation is not currently provided for the 3D advection case, this flag should be set to false for now

With n

NS: Use non_zero_time=false for problem type avection

Since exact solution and error calculation is not currently provided for the 3D advection case, this flag should be set to false for now

With non_zero_time=true the program was always reporting Max Error: 0. for problem type advection

show more ...

NS: Rename ICs_PetscMultiplicity() -> ICs_FixMultiplicity()

The name ICs_PetscMultiplicity was misleading, since the multiplicity is not related to PETSc

NS: Fix memory leak

Reported-by @jedbrown

Fluids - fix omitted indices correction in fluids version of CreateRestrictionFromPlex