-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add shape function DVs #202
Conversation
Codecov Report
@@ Coverage Diff @@
## main #202 +/- ##
==========================================
- Coverage 64.78% 64.76% -0.03%
==========================================
Files 47 47
Lines 11969 12018 +49
==========================================
+ Hits 7754 7783 +29
- Misses 4215 4235 +20
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This one is ready to be reviewed @hajdik, @sseraj. Maybe also @bernardopacini because you guys used this for the aviation paper. |
I added the MPhys wrapper function from my branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation changes look good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Purpose
This PR adds generic shape function DVs to the DVGeo class. This way, the users can specify shape functions that determine what FFD control point will move in what direction. A single shape function can affect multiple FFD points. A single FFD point can be affected by multiple shape functions; the displacements from each shape function are superposed. This also adds shape function DVs to the MPhys wrapper.
No tests yet, but wanted to create the PR to get the ball rolling.
Expected time until merged
2 weeks
Type of change
Testing
Checklist
flake8
andblack
to make sure the Python code adheres to PEP-8 and is consistently formattedfprettify
or C/C++ code withclang-format
as applicable