modulus-sym icon indicating copy to clipboard operation
modulus-sym copied to clipboard

🐛[BUG]: Issues in the NS implementation - Base NS equation class

Open ram-cherukuri opened this issue 1 year ago • 1 comments

Version

1.4.0

On which installation method(s) does this occur?

No response

Describe the issue

The current implementation of the base NS equation class has a few issues that result in incorrect flow behavior for a simple use case like flow over a cylinder in a channel.

  • xx
  • the use of the length scale brings in dimensional

Minimum reproducible example

No response

Relevant log output

No response

Environment details

No response

Other/Misc.

No response

ram-cherukuri avatar Feb 07 '24 19:02 ram-cherukuri

It is not possible to have a single Navier-Stokes equation class for all flow regimes as the nature of the equations changes. This will affect the convergence in training. Most common use case as seen in the documentation is incompressible flow with conjugate heat transfer. For incompressible flow, the continuity equation is inherently satisfied in the stress tensor. So the NS equations are simplified. This reduces the symbolic expressions being computed and allows the solution to propagate through the flow domain faster.

shourya-p-otta avatar Mar 04 '24 18:03 shourya-p-otta

The frameworks gives developers the flexibility to bring in their own formulation. We will add any new formulations of NS equation as needed for specific application use cases.

ram-cherukuri avatar Apr 10 '25 19:04 ram-cherukuri