moose icon indicating copy to clipboard operation
moose copied to clipboard

Add physics for fully coupled k-epsilon

Open GiudGiud opened this issue 9 months ago • 1 comments

TODO:

  • [ ] documentation
  • [ ] add energy wall function
  • [ ] add cross-parameter checking between BCs for energy & k-eps
  • [ ] add cross-parameter checking between wall momentum and k-eps BCs
  • [ ] expand testing reasonably. There are already 14 regression tests for turbulence trying out each wall function

It might take a bit to finish this. There's a lot, and a lot of options that don't work together. I ll get back to Components

GiudGiud avatar May 13 '24 23:05 GiudGiud

Job Documentation on f72b921 wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild avatar May 14 '24 16:05 moosebuild

Job Coverage on f72b921 wanted to post the following:

Framework coverage

1da09a #27614 f72b92
Total Total +/- New
Rate 85.04% 85.05% +0.00% 93.33%
Hits 105891 105904 +13 14
Misses 18622 18621 -1 1

Diff coverage report

Full coverage report

Modules coverage

Navier stokes

1da09a #27614 f72b92
Total Total +/- New
Rate 84.54% 84.59% +0.04% 83.87%
Hits 16356 16765 +409 468
Misses 2990 3055 +65 90

Diff coverage report

Full coverage report

Full coverage reports

Reports

Warnings

  • navier_stokes new line coverage rate 83.87% is less than the suggested 90.0%

This comment will be updated on new commits.

moosebuild avatar Jul 05 '24 20:07 moosebuild

Failure should be catchable in debug mode. I ll find it today or tomorrow, ready for a look otherwise

GiudGiud avatar Jul 22 '24 19:07 GiudGiud

@tanoret this should be ready for review!

After this we should decide whether we take an intermediate step of adding segregated-nonlinear support to the Physics, or create an entire set of Physics for the linear solver

GiudGiud avatar Jul 24 '24 13:07 GiudGiud

@tanoret if/when this looks good to you, I'll review/approve

lindsayad avatar Jul 31 '24 17:07 lindsayad

Thanks for the reviews!

GiudGiud avatar Sep 26 '24 01:09 GiudGiud

Seems it broke next on some distributed runs. I ll work on this now

GiudGiud avatar Sep 26 '24 12:09 GiudGiud