Closed pcarruscag closed 1 year ago
I'm not active in development anymore, but I do remember some of the history. Some of these decisions came from #797 . @economon specifically said (in that issue):
I removed the term involving the TKE from the viscous stress tensor in the mean flow equations, as this is often ignored in codes (and omitting it gives better results, including the pressure field). I also fixed a bug in the definition of the eddy viscosity, where strain magnitude was being used when it should be the vorticity magnitude.
You can see the motivation, regression testing, and explanations on that issue page.
The "Why ignore turb_ke?" comment originated in the refactoring effort of #1127 .
For a consistent implementation with SSTm (no tke in stress tensor) the production term should be P = mu_t S^2. Our production term is in no man's land, I'm suggesting we should complete the SST -> SSTm transition.
I also suggest to move to the 2003 version, where rate of strain is used in the eddy viscosity instead of vorticity.
And a few other changes in constants. I am absolutely fine with that but I do not have bandwidth to update all the regression tests, re-run validation tests, etc.
TLDR: There is SST and SSTm, the latter drops the tke term from the viscous stress tensor AND from the production term. Currently we do the former (because it was found beneficial here #797) but not the latter, which leaves the implementation in some kind of limbo state.
Discussed in https://github.com/su2code/SU2/discussions/1483