neko
neko copied to clipboard
Remove reduntant constructor in fluid
- Removes the
_uvwm
constructor. The code inside it overlapped with what was in the_all
, and both calledcommon
, so the contents of_uvw
could just be moved to common. - The
_all
essentially now deals with pressure. I think this should in principle be influid_pnpn
and it would be most reasonable to move it there in a subsequent PR. E.g. thespace
of pressure may be different. - Moved the code for the
output_bdry
to a separate subroutine - I moved most of the logging to the end of
_all
. This now resulted in that the log of GS comes before it says "Fluid"
--------Gather-Scatter--------
Comm : MPI
Avg. internal: 290672
Avg. external: 0
Backend : std
--------Gather-Scatter--------
Comm : MPI
Avg. internal: 16336
Avg. external: 0
Backend : std
--------Gather-Scatter--------
Comm : MPI
Avg. internal: 109424
Avg. external: 0
Backend : std
--------Gather-Scatter--------
Comm : MPI
Avg. internal: 560080
Avg. external: 0
Backend : std
--------Gather-Scatter--------
Comm : MPI
Avg. internal: 290672
Avg. external: 0
Backend : std
------------Fluid-------------
Type : Modular (Pn/Pn)
Ksp prs. : (gmres, hsmg)
`-abs tol : 1.000000E-07
Ksp vel. : (cg, jacobi)
`-abs tol : 1.000000E-07
rho : 1.000000E+00
mu : 7.142857E-04
Dealias : T
Save bdry : T
Since we mostly reuse the GS I think this is fine, but I can move the header back if needed.