Skip to content

Unsteady incompressible flow simulation with grid movement - #767

Merged
rsanfer merged 51 commits into
developfrom
feature_incompressible_ale
Sep 5, 2019
Merged

Unsteady incompressible flow simulation with grid movement#767
rsanfer merged 51 commits into
developfrom
feature_incompressible_ale

Conversation

@cvencro

@cvencro cvencro commented Aug 22, 2019

Copy link
Copy Markdown
Contributor

Proposed Changes

This request enables unsteady incompressible flow simulation with grid movement. Changes are made as part of the collaboration with @economon and @TobiKattmann to expand the capabilities of the incompressible solver. Arbitrary Lagrangian-Eulerian formulation of the Low-Mach Navier-Stokes equations have been implemented which is suitable for both rigid grid motion and mesh deformation problems.

Regression test case to follow.

Related Work

#766

PR Checklist

  • [X ] I am submitting my contribution to the develop branch.
  • [X ] My contribution generates no new compiler warnings (try with the '-Wall -Wextra -Wno-unused-parameter -Wno-empty-body' compiler flags).
  • [X ] My contribution is commented and consistent with SU2 style.
  • [X ] I have added a test case that demonstrates my contribution, if necessary.

cvencro and others added 30 commits September 26, 2018 22:35
… feature_incompressible_ale

Conflicts:
	Common/src/config_structure.cpp
	Common/src/geometry_structure.cpp
Conflicts:
	Common/src/grid_movement_structure.cpp
	SU2_CFD/src/output_su2.cpp
@talbring

talbring commented Sep 2, 2019

Copy link
Copy Markdown
Member

@TobiKattmann Just test whether it works and gives the same results. The regression test should be added still with the old driver. I will adapt it accordingly.

@TobiKattmann

TobiKattmann commented Sep 3, 2019

Copy link
Copy Markdown
Contributor

@cvencro & @talbring, Ok Testcase added using the old driver for serial and parallel regression. Residuals are identical for both drivers new & old (which should resolve your question @rsanfer, right?). In the Testcase .cfg the new driver options are commented out such that switching is a matter of (un)commenting lines.

I put up a PR for the Testcase here su2code/TestCases#36 and if the regression tests pass I would merge the Testcase PR and revert the .travis.yml

Update: su2code/Testcases is merged. So once my comment above concerning rotating frame is addressed this could be merged from my point of view.

@talbring
talbring requested a review from rsanfer September 3, 2019 14:45
@cvencro

cvencro commented Sep 3, 2019

Copy link
Copy Markdown
Contributor Author

@TobiKattmann , thanks for the regression test. The unsteady grid motion simulations work with the updates to the ALE formulation. The rotating frame is solved as a steady problem with an added source term, an initial implementation of which has been implemented for the incompressible solver but that is still a work in progress and probably outside the scope of this PR as it stands, so I've added an error catch.

@TobiKattmann

Copy link
Copy Markdown
Contributor

Ok, I now believe that all issues/comments are addressed.
@rsanfer , the SINGLEZONE DRIVER check is the only remaining requested change currently. @cvencro and I both confirmed that the residuals are identical to the old driver.
So if you mark that as resolved and there are no further comments, I think this can be merged :)

@economon economon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changed and test case! LGTM.

@rsanfer

rsanfer commented Sep 5, 2019

Copy link
Copy Markdown
Contributor

Yep, I think this is good to go. Thanks, @cvencro and @TobiKattmann!!

@rsanfer
rsanfer merged commit 1c685ff into develop Sep 5, 2019
@rsanfer
rsanfer deleted the feature_incompressible_ale branch September 5, 2019 08:06
@talbring talbring changed the title Feature incompressible ale Unsteady incompressible flow simulation with grid movement Nov 8, 2019
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.

6 participants