Fix sign error in incompressible Spalding wall function#2541
Fix sign error in incompressible Spalding wall function#2541bigfooted merged 5 commits intosu2code:developfrom
Conversation
Co-authored-by: Nijso <nijso@hotmail.com>
|
The regression tests for the wall functions fail now and have to be updated. Can you do this? You have to go to Testcases/serial_regression.py inc_turb_wallfunction_flatplate_sa.test_vals = [-6.894206, -5.715970, -6.743740, -4.242551, -9.550272, 10.000000, -2.879370, 0.001021, 0.003759] Same for: inc_turb_wallfunction_flatplate_sst.test_vals = [-6.881306, -5.729111, -6.724803, -4.242636, -7.162846, -2.044959, 10.000000, -2.877924, 0.001151, 0.003161, 0.000000] and in Testcases/Tutorials.py: sudo_tutorial.test_vals = [-14.664419, -12.789769, -13.280336, -13.016392, -13.018817, -9.510066, 15.000000, -1.994192] |
|
I've completed all the updates you requested. Let me know if anything else needs adjusting |

Proposed Changes
Fix a sign error in the Spalding wall function term in INC_NSSolver.cpp. The constant '+1.0' is changed to '-1.0' to match Spalding(1961) and Nichols & Nelson (2004). I've also checked the derivatives, but there were no issues.
Related Work
PR Checklist
Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.
pre-commit run --allto format old commits.