Closed sseraj closed 2 years ago
Merging #154 (4ceb66c) into main (787a96d) will increase coverage by
0.00%
. The diff coverage is73.33%
.
@@ Coverage Diff @@
## main #154 +/- ##
=======================================
Coverage 63.89% 63.90%
=======================================
Files 47 47
Lines 11745 11757 +12
=======================================
+ Hits 7505 7513 +8
- Misses 4240 4244 +4
Impacted Files | Coverage Δ | |
---|---|---|
pygeo/parameterization/DVGeoMulti.py | 89.77% <73.33%> (-0.23%) |
:arrow_down: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Purpose
I made two changes related to
addPointSet
kwargs for DVGeometryMulti: 1) Added an option to pass kwargs for the triangulated surface mesh 2) Pass kwargs from DVGeometryMulti'saddPointSet
to the componentaddPointSet
I also changed mutable default assignments to be inside functions. These were not causing any issues to my knowledge, but this should be safer.
Expected time until merged
1 week
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