-
Notifications
You must be signed in to change notification settings - Fork 904
Implementation of DDES and SAS formulations for SST #2150
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
Open
rois1995
wants to merge
68
commits into
develop
Choose a base branch
from
feature_SST_IDDES
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.
+527
−111
Open
Changes from 67 commits
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
388f85a
- Added computation of desired grid size for Scale Resolving Simulati…
41cb27e
- Added a more complex SAS model named SAS_COMPLICATED
a6be2cf
- Fixed getting coordinates of points
d1f4db6
- Added output of laplacian
35958b8
- Fixed missing variable initialization for CTurbSAVariable
09e977f
- Added DES implementations to SST: DDES, Improved DDES (IDDES), Simp…
8c8e89d
- Added SST DDES in config
71ac9c4
- Added Debug quantities
5b5fc30
- Added division by 0 protection
ef50283
- Added computation of LESIQ to Output
91c742e
Merge branch 'develop' into feature_SST_IDDES
rois1995 6c86a19
Merge 'origin/feature_SAS' into feature_SST_IDDES
48a9f3d
- modified codi
ae33b05
- Modified Velocity Laplacian (vector -> Matrix)
6443070
- Added outputs log for SST-DDES
caf0d41
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
a584526
- update externals
0f3a832
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
710e1dc
Fix SAS Babu implementation
e4f28d4
- Fixed compilation error
068ac21
- Moved output functions not to brake the restart
cbe79f7
- Added limiting to eddy viscosity (needed?)
d770fa1
Update SU2_CFD/include/numerics/CNumerics.hpp
rois1995 4dda141
Update SU2_CFD/include/numerics/CNumerics.hpp
rois1995 84d65bf
- change grid size computation to account for 2D
5337229
- Added output of debug variables to SAS
0806d21
- Fixed bug that always activates SAS Babu
0cab1ef
- Modified the C_S constant in Lvk limiter
ad86192
Changed computation of Laplacian
dcbacc8
- slight changes to functions calls
33c6664
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
a5702d8
- align externals and submodules to develop
ab6bedb
-fixed compilation errors
e5080a1
- Solve declaration hids variable
c2fd653
- SRS Grid size output (fix?)
d377903
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
d037e59
- Added SLA DDES formulation to SST
d9f329e
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
0adcb7e
- compilation erorr
d50b123
- Fix memory leak
34a0271
- fixed formatting error
04dfcd3
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
a60fb13
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
95ccc73
- update externals
de45d5f
- update externals and subprojects
d172607
- Try changes to SA-EDDES
e5a916c
- Fix bug in SST-EDDES
c723a13
- Set debug quantities
d051f0b
- try fix
c840256
- Fix output
9b997d7
- Fix output
8f3efae
- Added come outputs
f716218
- minor fix
28490fd
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
ff56f36
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
707afa4
update meson and ninja
b249f2c
Merge remote-tracking branch 'origin/develop' into HEAD
f10c465
Removed SAS implementation (not working)
37a866f
- Removed debug quantities
ac2078c
- Re-introduced the virtual function
0c8d49a
- Fixed some notation
86db90b
- Fix compilation issues
d4564da
- Fixed variable inputs
177276a
- Fixed some thread sanitizer problems
e260cc4
- Removed unused functions
7e57afe
- Fixed some notation
f193796
- Added SST-EDDES formulation for unstructured grids
c5e1dee
Merge remote-tracking branch 'origin/develop' into feature_SST_IDDES
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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
Are the constant here also used somewhere else? It would be good to avoid repeating hard-coded constants
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.
They are used just here