-
Notifications
You must be signed in to change notification settings - Fork 842
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
[WIP] Implementation of Simplified LM Transition model #1901
Open
rois1995
wants to merge
22
commits into
develop
Choose a base branch
from
feature_Trans_SLM
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,255
−135
Open
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
a7136d3
- Just copied the CTransLMSolver header and source to commit
rois1995 e9226db
- Start of implementation of Simplified LM transition model
rois1995 d03742a
- Modify number of variables if SLM option is used
rois1995 e17872d
- Updates on the transition model.
rois1995 4b9c615
- Added Cross flow effects for SLM
rois1995 04cd56d
- Fixed Intermittency_Sep computation for SLM.
rois1995 e29b4df
- Added computation for normals of grid points
rois1995 0319d62
Fixed normal extraction from structure
rois1995 b1edf61
Added computation of wall normals to the CPoint structure
rois1995 7eee4ab
Check For changes
rois1995 c461518
Removing changes for SA-R
rois1995 e68a9e4
Fixed vertex indexing for wall normal computation
rois1995 bcdd671
Removed a cout
rois1995 9bb2435
Fixed division by zero with Corr_Rec
rois1995 a9a6ce4
Removed cout
rois1995 86b068c
Fixed Max Velocity-Z output for Incompressible flow
rois1995 73c0a4b
Fixed output of Normals in volume
rois1995 8d5148e
- Added Simplified Langtry Menter model
rois1995 9db7301
- corrected bu in output
rois1995 1e7e7a3
Merge branch 'develop' into feature_Trans_SLM_v8
rois1995 ab2d71b
Merge remote-tracking branch 'origin/feature_Trans_SLM_v8' into featu…
rois1995 a6fe9bc
- finish update
rois1995 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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3495,9 +3495,9 @@ void CConfig::SetPostprocessing(SU2_COMPONENT val_software, unsigned short val_i | |
if (Kind_Trans_Model == TURB_TRANS_MODEL::LM) { | ||
lmParsedOptions = ParseLMOptions(LM_Options, nLM_Options, rank, Kind_Turb_Model); | ||
|
||
/*--- Check if problem is 2D and LM2015 has been selected ---*/ | ||
if (lmParsedOptions.LM2015 && val_nDim == 2) { | ||
SU2_MPI::Error("LM2015 is available only for 3D problems", CURRENT_FUNCTION); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is LM2015 gone? or is crossflow the same? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have changed the option to CROSSFLOW, since I will use it also for the Simplified model. |
||
/*--- Check if problem is 2D and CrossFlow has been selected ---*/ | ||
if (lmParsedOptions.CrossFlow && val_nDim == 2) { | ||
SU2_MPI::Error("Cross-flow corrections are available only for 3D problems", CURRENT_FUNCTION); | ||
} | ||
} | ||
|
||
|
@@ -6230,11 +6230,27 @@ void CConfig::SetOutput(SU2_COMPONENT val_software, unsigned short val_izone) { | |
switch (Kind_Trans_Model) { | ||
case TURB_TRANS_MODEL::NONE: break; | ||
case TURB_TRANS_MODEL::LM: { | ||
cout << "Transition model: Langtry and Menter's 4 equation model"; | ||
if (lmParsedOptions.LM2015) { | ||
cout << " w/ cross-flow corrections (2015)" << endl; | ||
int NTurbEqs = 0; | ||
switch (Kind_Turb_Model) { | ||
case TURB_MODEL::SA: NTurbEqs = 1; break; | ||
case TURB_MODEL::SST: NTurbEqs = 2; break; | ||
case TURB_MODEL::NONE: SU2_MPI::Error("No turbulence model has been selected but LM transition model is active.", CURRENT_FUNCTION); break; | ||
} | ||
if (!lmParsedOptions.SLM) { | ||
int NEquations = 2; | ||
cout << "Transition model: Langtry and Menter's "<< NEquations+NTurbEqs <<" equation model"; | ||
} else { | ||
cout << " (2009)" << endl; | ||
int NEquations = 1; | ||
cout << "Transition model: Simplified Langtry and Menter's "<< NEquations+NTurbEqs <<" equation model"; | ||
} | ||
if (lmParsedOptions.CrossFlow) { | ||
cout << " w/ cross-flow corrections" << endl; | ||
} else { | ||
if (!lmParsedOptions.SLM) { | ||
cout << " (2009)" << endl; | ||
} else { | ||
cout << " (2015)" << endl; | ||
} | ||
} | ||
break; | ||
} | ||
|
@@ -6258,6 +6274,13 @@ void CConfig::SetOutput(SU2_COMPONENT val_software, unsigned short val_izone) { | |
} | ||
break; | ||
} | ||
cout << "Correlation Functions for Simplified LM model: "; | ||
switch (lmParsedOptions.Correlation_SLM) { | ||
case TURB_TRANS_CORRELATION_SLM::CODER_SLM: cout << "Coder et al. (2012)" << endl; break; | ||
case TURB_TRANS_CORRELATION_SLM::MOD_EPPLER_SLM: cout << "Modified Eppler (from Coder et al. 2012)" << endl; break; | ||
case TURB_TRANS_CORRELATION_SLM::MENTER_SLM: | ||
case TURB_TRANS_CORRELATION_SLM::DEFAULT: cout << "Menter et al. (2015)" << endl; break; | ||
} | ||
} | ||
cout << "Hybrid RANS/LES: "; | ||
switch (Kind_HybridRANSLES) { | ||
|
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Can you add the param[out] for these please
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.
Sure, I'll do it now.