Fix reference pressure for Cp with moving frames (e.g. ROTATING_FRAME) by pcarruscag · Pull Request #1595

Fix reference pressure for Cp with moving frames (e.g. ROTATING_FRAME) by pcarruscag · Pull Request #1595 · su2code/SU2

Merged

pcarruscag merged 4 commits intodevelopfrom

fix_1588

Apr 11, 2022

Merged

Fix reference pressure for Cp with moving frames (e.g. ROTATING_FRAME)#1595
pcarruscag merged 4 commits intodevelopfrom
fix_1588

Conversation

Copy link

Member

pcarruscag commented
Apr 6, 2022
•

edited

Loading

Fixes #1588
cc @ArneVoss
I have not fully tested the python wrapper forces, but they should be ok for dimensional simulations.
This is WIP until I add a regression test.

Use the ref dynamic pressure of the solver to compute Cp, different w…

c2018c8

…ay to re-dim vertex tractions

pcarruscag added the changelog:fix label

Apr 6, 2022

add a regression

9ec0c58

pcarruscag changed the title ~~[WIP] Fix reference pressure for Cp with moving frames (e.g. ROTATING_FRAME)~~ Fix reference pressure for Cp with moving frames (e.g. ROTATING_FRAME)

Apr 9, 2022

pr-triage bot added the PR: unreviewed label

Apr 9, 2022

Copy link

Member Author

pcarruscag commented
Apr 9, 2022

python forces are also ok for non-dimensional simulations

pcarruscag added 2 commits

April 9, 2022 19:35

build pywrapper for hybrid tests

5cfa1de

adjust regression values

4ccf9cd

Copy link

Member

ArneVoss commented
Apr 11, 2022

Pedro,
I cherry-picked your your commit c2018c8 and tested it with my configuration. Looks good so far, I haven't see any issues / problems on my side. Thank you very much!

pcarruscag merged commit 9494cbd into develop

Apr 11, 2022

pcarruscag deleted the fix_1588 branch

April 11, 2022 19:10

pr-triage bot added PR: merged and removed PR: unreviewed labels

Apr 11, 2022

ArneVoss mentioned this pull request