-
Notifications
You must be signed in to change notification settings - Fork 52
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 composite DVs #119
Add composite DVs #119
Conversation
Codecov Report
@@ Coverage Diff @@
## master #119 +/- ##
==========================================
- Coverage 63.86% 63.40% -0.47%
==========================================
Files 41 41
Lines 10931 11033 +102
==========================================
+ Hits 6981 6995 +14
- Misses 3950 4038 +88
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
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.
Nice stuff, most of my comments are to get a better understanding about what is going on.
Two comments:
- Shall we rename
convertSensitivitytoDict
and the reverse one? it seems it's of more general use than for just sensitivities - I assume this is done by the new map functions at the end of
DVGeo.py
, but how are the DVs mapped back to the user at the end?
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.
Good stuff. I have few minor comments, but good to go otherwise
pygeo/parameterization/DVGeo.py
Outdated
@@ -1237,6 +1239,30 @@ class in geo_utils. Using pointSelect discards everything in volList. | |||
|
|||
return self.DV_listSectionLocal[dvName].nVal | |||
|
|||
def addGeoDVComposite(self, dvName, ptSetName=None, u=None, scale=None): |
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.
Please add some comments or dosctrings briefly explaining that this is more of a preprocessing routine thats run on a single proc.
Purpose
This PR adds composite DVs, which are linear combinations of existing DVs.
Expected time until merged
Somewhat urgent, by end of the week please.
Type of change
Testing
Tests will come at a later time.
Checklist
flake8
andblack
to make sure the code adheres to PEP-8 and is consistently formatted