Exawind / nalu-wind

Solver for wind farm simulations targeting exascale computational platforms
https://exawind.github.io/nalu-wind/
Other
125 stars 85 forks source link

Capability deprecation in nalu-wind #498

Closed rcknaus closed 2 months ago

rcknaus commented 4 years ago

Here's a short list of things that we can consider getting rid of in nalu-wind. Not really meant as a full list or anything---I just want to start off the discussion.


Percept-based, dynamic mesh adaptation: What is it: conformal refinement of meshes given a chosen error estimator during a simulation

Two-dimensional support What is it: mesh data structures that are truly two-dimensional

Participating media radiation What is it: An equation system describing how thermal radiation is emitted and reabsorbed in a fluid

Mass Fraction What is it: An equation describing multicomponent flows with potentially differential diffusion evolve

Conjugate Heat Transfer (CHT) What is it: Coupled heat transfer between a fluid and a solid

Realms What is it: a conceptualization of different physics or intents

sayerhs commented 4 years ago

@rcknaus Thanks for starting this list.

I agree with deprecating Percept-based mesh adaptivity (already removed in NaluCFD/Nalu), PMR, CHT, and Flamelets stuff. It will be there in the git-history if we need to dig these up.

Regarding 2D, most of the element types have been migrated to NGP. However, it would be great to be able to make ndim a compile time constexpr in several places.

Regarding Mass/Mixture fraction, what are your thoughts on the whole offshore wind-wave simulation needs and the use of these equation systems for that purpose?

Our FSI pathway by coupling with OpenFAST doesn't use multiple-realms, so Realms is just a glorified std::vector that could easily live in TimeIntegrator.

rcknaus commented 4 years ago

For offshore, I don't know the plan, but saying it's some level-set/volume of fluid variant then the solution process is different enough that we probably just want it to have its own equation system anyway.

rcknaus commented 2 months ago

Most of these things have been removed and removing the remaining set is of dubious benefit.