-
Notifications
You must be signed in to change notification settings - Fork 849
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleanup wall functions and add user controls #1360
Merged
Merged
Changes from 1 commit
Commits
Show all changes
125 commits
Select commit
Hold shift + click to select a range
e5e5d19
fixed wall functions for SA and SST
bigfooted 6be5670
cleaning up wall model implementation
bigfooted a62a0b8
cleaning up wall model implementation
bigfooted ecc7d3f
cleaning up wall model implementation
bigfooted f643214
Merge branch 'develop' into fix_wallfunctions
bigfooted 3705941
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted 25fd773
merge develop back in
bigfooted cc1798f
cleanup based on code review
bigfooted e17af6d
Merge branch 'fix_wallfunctions' of https://github.com/su2code/SU2 in…
bigfooted 76a14b2
Merge branch 'develop' into fix_wallfunctions
pcarruscag dbde0f7
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted 8e4c4b8
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted f7972ae
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.hpp
bigfooted b15e5f9
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted f3551d8
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted 51da480
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted f3f7139
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted de8f6cd
Update SU2_CFD/src/solvers/CTurbSSTSolver.cpp
bigfooted ae7687e
Update SU2_CFD/include/solvers/CIncNSSolver.hpp
bigfooted 7b61985
small update
bigfooted 3e996bd
Merge branch 'fix_wallfunctions' of https://github.com/su2code/SU2 in…
bigfooted 9437a11
Merge branch 'develop' into fix_wallfunctions
bigfooted 658e7cb
Update SU2_CFD/include/solvers/CSolver.hpp
bigfooted ae47196
fix unused variables update
bigfooted 06898d6
Merge branch 'fix_wallfunctions' of https://github.com/su2code/SU2 in…
bigfooted 69e7fe1
fix unused variables update
bigfooted 6d629c8
fix wall model constant config option
bigfooted 0c0e3e6
fixed alignment and missing break statement
bigfooted fa88243
computing skinfriction when y+ < 5
bigfooted db32076
Merge remote-tracking branch 'upstream/develop' into develop
bigfooted 4a4549b
merge with develop
bigfooted 995f587
fix unused beta_1
bigfooted f713575
Merge remote-tracking branch 'upstream/develop' into develop
bigfooted e3b34a8
update with develop
bigfooted 4b82974
fix OMP command
bigfooted 423a89e
fix OMP command
bigfooted a6dc387
update to enum class
bigfooted b2fb98c
medi/ninja
bigfooted e32750f
remove vscode
bigfooted c33db7c
Update SU2_CFD/include/solvers/CTurbSSTSolver.hpp
pcarruscag 722f3bb
Merge branch 'hybrid_parallel_ad3' into fix_wallfunctions
pcarruscag dabd9ee
Apply suggestions from code review
pcarruscag af96967
Merge branch 'fix_wallfunctions' of https://github.com/su2code/SU2 in…
pcarruscag a626425
small cleanup
pcarruscag ea38fd5
Merge remote-tracking branch 'upstream/develop' into fix_wallfunctions
pcarruscag a5a5835
a little more refactoring
pcarruscag fb3c0cc
apply fix from #1120 (do not consider tau wall for edges along wall),…
pcarruscag 4172408
added config options
bigfooted 44e5126
fixing wall temperature for wall function
557b950
Merge remote-tracking branch 'upstream/develop' into fix_wallfunctions
pcarruscag 90723af
some fixes and better warnings
pcarruscag 7ab705c
Merge branch 'develop' into fix_wallfunctions
pcarruscag 9fd30a9
add testcase
bigfooted 7d0d172
add testcase
bigfooted 2b371f8
add config options for wall functions
bigfooted b874239
Merge branch 'su2code:fix_wallfunctions' into fix_wallfunctions
bigfooted 3306eb9
fix y+ bug
bigfooted 8c0a1f2
added testcase
bigfooted aa009e0
Merge branch 'develop' into fix_wallfunctions
bigfooted 3a68a3e
remove deleted files CDiscAdFEAVariable
bigfooted 075ef48
update wall temperature for aerodynamic heating
bigfooted caf9ab7
remove unused variables
bigfooted 8066a23
Update Common/src/CConfig.cpp
bigfooted 3ff82e5
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted ec5ecec
Update Common/include/CConfig.hpp
bigfooted 67b1a53
Update TestCases/wallfunctions/flatplate/turb_SA_flatplate.cfg
bigfooted 1591c7c
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted de13ceb
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted f7d3a6b
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted 2d76265
reviewer changes
bigfooted 9a67668
Merge branch 'fix_wallfunctions' of https://github.com/bigfooted/SU2 …
bigfooted d7197e0
update testcase
bigfooted 9f39f7d
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted ea14c02
Update SU2_CFD/include/solvers/CFVMFlowSolverBase.inl
bigfooted 4226dbf
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted b00e0ec
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted 5af6111
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted b268a9d
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 73f456c
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted b2db011
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 0940d87
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 238c9e5
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 474cee3
Update SU2_CFD/src/solvers/CTurbSASolver.cpp
bigfooted 310585b
Update SU2_CFD/src/solvers/CTurbSSTSolver.cpp
bigfooted 34a6eff
Update SU2_CFD/src/solvers/CTurbSSTSolver.cpp
bigfooted 322f010
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 10a69be
change some comments
bigfooted 7c9b447
Update TestCases/wallfunctions/flatplate/turb_SA_flatplate.cfg
bigfooted 3f4c2d0
Update TestCases/wallfunctions/flatplate/turb_SA_flatplate.cfg
bigfooted 827fbbc
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted ec87ed9
add additional testcases
bigfooted ffba868
Merge branch 'develop' into fix_wallfunctions
bigfooted dfd2939
added incomp. and comp. testcases
bigfooted 545f139
cleanup of testcases config files
bigfooted 00548d9
Update SU2_CFD/src/solvers/CIncNSSolver.cpp
bigfooted 7f5a342
removed logfile
bigfooted 0ca421f
Update SU2_CFD/src/solvers/CNSSolver.cpp
bigfooted 1dd5c96
added nondimensionalization
bigfooted bbca88c
Merge branch 'fix_wallfunctions' of https://github.com/bigfooted/SU2 …
bigfooted 11cb94e
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann fc03901
Changes to Testcases cfg's. Fix convergence for inc cases.
TobiKattmann 437d1fc
Add '_' in some var names to be consistent with cfg.
TobiKattmann 651d669
Remove unused Wall_Functions Marker from Testcases repo.
TobiKattmann e48ba40
Remove empty added line.
TobiKattmann 578822e
Rename TauWall functions to Tau_Wall to be consistent with var-name.
TobiKattmann 3daef85
Update reg test vals. Adding more screen output fields.
TobiKattmann e2cdf26
Merge branch 'fix_wallfunctions' of https://github.com/bigfooted/SU2 …
TobiKattmann 5a166b6
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann dde0a9f
Remove unnecessary nondim. All values come from the solver itself.
TobiKattmann 194692c
Const-ing vars in SetTau_Wall_WF func for NS and IncNS.
TobiKattmann 9fe38b7
Rename Conductivity_Ref -> Thermal_Conductivity_Ref.
TobiKattmann 0af828b
else not needed due to continue
pcarruscag d3b2fed
Minor changes. Comments and moved variable.
TobiKattmann b0bcd88
Merge branch 'fix_wallfunctions' of https://github.com/bigfooted/SU2 …
TobiKattmann 7a7194c
Merge branch 'develop' into fix_wallfunctions
pcarruscag 7d0a480
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann 17e52ea
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann dd8d981
do not recompute y+ when wall functions are used
bigfooted b9e83e7
Update cfg files. Cleaning + formatting.
TobiKattmann e35cfeb
Merge branch 'develop' into fix_wallfunctions
TobiKattmann af850b9
Update TestCases/serial_regression.py
pcarruscag 0111ebb
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann 9a3804a
Merge branch 'fix_wallfunctions' of https://github.com/bigfooted/SU2 …
TobiKattmann f9a5129
Merge remote-tracking branch 'origin/develop' into fix_wallfunctions
TobiKattmann d492fb4
Do some checks for the wall function impl in CConfig.
TobiKattmann File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this "final" addition and all the help on this (where of course @bigfooted did the heavy lifting ;) )
Now this is good to go from my perspective, I have no further comments. Maybe we can merge this in todays dev meeting :)
Things tbd in the future are
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright merging this in now✈️