SU2 icon indicating copy to clipboard operation
SU2 copied to clipboard

Axisymmetry for SA is not implemented

Open bigfooted opened this issue 2 years ago • 7 comments

Describe the bug A clear and concise description of what the bug is and what you expect the behavior to be instead. If applicable, add screenshots to help explain your problem.

2D axisymmetry source terms for the SA turbulence model are not implemented. There is also no warning/error to the user.

bigfooted avatar Mar 18 '22 21:03 bigfooted

An error for SA+axissymmetry is introduced in #1571. So this then now labels as a feature-request :)

TobiKattmann avatar Mar 24 '22 09:03 TobiKattmann

If anyone has a source for this, Id be interested in adding this.

WallyMaier avatar Apr 13 '22 01:04 WallyMaier

@WallyMaier I don't have a reference showing the specific terms for SA, but you can derive it from the general transformation from Cartesian to cylindrical coordinates. It looks like the only term that is affected is the diffusion term. The appendix of Transport Phenomena (Bird, Stewart, Lightfoot) shows some Cartesian and cylindrical forms for a general scalar transport equation. The result is that the divergence of a flux in Cartesian coordinates is $$(1/r) * (d/dr)(rJ_x) + (1/r)(d/d theta)(J_{theta}) + d/dz(J_z)$$ and we neglect the theta-term in axisymmetric flow.

bigfooted avatar Apr 16 '22 11:04 bigfooted

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is still a relevant issue please comment on it to restart the discussion. Thank you for your contributions.

stale[bot] avatar Nov 01 '22 23:11 stale[bot]

I'm working on this. https://github.com/su2code/SU2/tree/feature_SA_axisymmetry

bigfooted avatar Nov 02 '22 08:11 bigfooted

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is still a relevant issue please comment on it to restart the discussion. Thank you for your contributions.

stale[bot] avatar Jan 07 '23 19:01 stale[bot]