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

Discrete adjoint for deforming meshes #833

Merged
merged 135 commits into from
Apr 10, 2020
Merged

Conversation

cvencro
Copy link
Contributor

@cvencro cvencro commented Dec 8, 2019

Proposed Changes

This PR introduces surface deformation routines in CMeshSolver and enables discrete adjoint calculations for deforming mesh problems. Gradient verification for a pitching airfoil using a deforming mesh is good compared against rigid motion for the same pitching airfoil:

image

Related Work

Contains PR #919 which cleaned up legacy FSI and deformation methods
Builds on work previously discussed in PR #766
Includes fix for issue #860

PR Checklist

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with the '-Wall -Wextra -Wno-unused-parameter -Wno-empty-body' compiler flags).
  • My contribution is commented and consistent with SU2 style.
  • 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.

@pr-triage pr-triage bot added the PR: draft label Dec 8, 2019
@cvencro cvencro changed the title Discrete adjoint for deforming meshes WIP Discrete adjoint for deforming meshes Dec 8, 2019
@cvencro cvencro mentioned this pull request Feb 17, 2020
5 tasks
Copy link
Member

@pcarruscag pcarruscag left a comment

Choose a reason for hiding this comment

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

LGTM
Contains the work from #914 and #919.

@pcarruscag pcarruscag merged commit 55a70bf into develop Apr 10, 2020
@pcarruscag pcarruscag deleted the feature_dyn_deform_adjoint branch April 10, 2020 19:18
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.

2 participants