Several changes to demoDesignVars

[sseraj]

Purpose

I made several changes to demoDesignVars, mainly:

• Added an option to write out the deformed surface mesh by passing in an ADflow object
• Made having a point set optional. If you only want the deformed FFD, there is no need for DVGeo to have a point set.
• Fixed the initialization of the upper variable. The function now works if a parent FFD has no DVs.
• Changed the file naming to reset the counter for every DV instead of having one counter for all DVs

Type of change

• [x] Bugfix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (non-backwards-compatible fix or feature)
• [ ] Code style update (formatting, renaming)
• [ ] Refactoring (no functional changes, no API changes)
• [ ] Documentation update
• [ ] Maintenance update
• [ ] Other (please describe)

Testing

In your optimization script, before running the optimizer, put something like the following lines:

CFDSolver.setAeroProblem(ap)
DVGeo.demoDesignVars(outDir, CFDSolver=CFDSolver)

Checklist

• [x] I have run flake8 and black to make sure the code adheres to PEP-8 and is consistently formatted
• [x] I have run unit and regression tests which pass locally with my changes
• [ ] I have added new tests that prove my fix is effective or that my feature works
• [x] I have added necessary documentation

[codecov[bot]]

Codecov Report

Merging #104 (ab00203) into master (435d7ad) will increase coverage by 0.67%. The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
+ Coverage   59.48%   60.16%   +0.67%     
==========================================
  Files          40       40              
  Lines        9614     9617       +3     
==========================================
+ Hits         5719     5786      +67     
+ Misses       3895     3831      -64     

Impacted Files	Coverage Δ
pygeo/parameterization/DVGeo.py	65.51% <72.22%> (+3.41%)	:arrow_up:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 435d7ad...ab00203. Read the comment docs.

[sseraj]

We don't want to test this with a CFDSolver instance... maybe we can add that test to the MACH-Aero tutorial?

Right, I didn't want to introduce ADflow as a testing dependency. Given that this only tests the files created, I don't see much benefit in testing this in MACH-Aero.