Skip to content
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

Corrections to SA-NEG-R and SA model #2014

Merged
merged 29 commits into from
Jun 10, 2023
Merged

Corrections to SA-NEG-R and SA model #2014

merged 29 commits into from
Jun 10, 2023

Conversation

rois1995
Copy link
Contributor

@rois1995 rois1995 commented Apr 20, 2023

Proposed Changes

Hi, I found some problems with the implementation of the SA-NEG and SA-NEG-R models wrt to what is described on TMR. Plus, I found a suggestion on the computation of the SHat variable to avoid clipping for negative values (also reported on TMR).

Related Work

  • Implementation of new computation of SHat (Jacobian was computed by hand and is somewhat complex. Probably I could derive a simplified formulation)
  • Added CRot constant for SA-R model
  • Modified computation of var.S for the combination SA-NEG-R
  • Slightly change the computation of muT from nu_tilde: now muT = 0 if nu_tilde <0. Before it was muT = max(muT, 0.0), which should not always coincide with nu_tilde > 0.
  • Not related to SA: added viscous drag as output in the history file.

I should also try to add the maximum value of nu_tilde within the mesh to the history file, although I still have to implement it.

PR Checklist

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
  • My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
rois1995 and others added 4 commits April 21, 2023 09:06

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@pcarruscag
Copy link
Member

Can you guys wrap this up and update the required restart files?
I want to make the fixes to the SST model we want for v8.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@rois1995
Copy link
Contributor Author

@pcarruscag Sorry for the delay. Which restart files are you referring to?

@pcarruscag
Copy link
Member

Some of the testcases that show as failed need to be updated, if the update is significant for cases with restart files we need a new restart file to match the previous convergence level.

bigfooted added 2 commits May 24, 2023 21:04

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@bigfooted
Copy link
Contributor

I just updated the serial and parallel regression tests. There is one SA test that has a restart file, I will also update that one.
@rois1995 can you have a look at the failed cases for the tutorial.py and vandv.py regression tests? I will have a look at the *_AD.py regressions this Friday.

@rois1995
Copy link
Contributor Author

@bigfooted sure, I'll look into it right now.

@pcarruscag pcarruscag changed the title [WIP] Corrections to SA-NEG-R and SA model Corrections to SA-NEG-R and SA model Jun 3, 2023
pcarruscag and others added 9 commits June 3, 2023 21:25

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@bigfooted bigfooted merged commit 536def1 into develop Jun 10, 2023
@bigfooted bigfooted deleted the feature_SA_HLPW5 branch June 10, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants