Closed lucaeros closed 10 months ago
Please add sudo apt-get update
to the beginning of the Install OpenVSP
step in the GitHub Actions workflow. Hopefully this will fix the 404 error preventing the actions from properly installing packages. Including the apt update is noted here, which is linked to from a GitHub issue about a similar problem.
Merging #413 (e95326d) into main (5ebabd1) will increase coverage by
0.00%
. The diff coverage is98.61%
.
@@ Coverage Diff @@
## main #413 +/- ##
=======================================
Coverage 94.50% 94.50%
=======================================
Files 103 103
Lines 6445 6448 +3
=======================================
+ Hits 6091 6094 +3
Misses 354 354
Files Changed | Coverage Δ | |
---|---|---|
openaerostruct/geometry/geometry_mesh.py | 98.63% <83.33%> (-0.06%) |
:arrow_down: |
openaerostruct/__init__.py | 100.00% <100.00%> (ø) |
|
...rostruct/geometry/geometry_mesh_transformations.py | 99.67% <100.00%> (+<0.01%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Please pull the latest main to fix the build and also bump the version to 2.7.0
Purpose
Following the previous PR on chord_scaling_pos, the aim of this PR is to add a new setting called "ref_axis_pos" (stands for reference axis position) that is generalizing the "chord_scaling_pos" for every geometric transformation that needed it : twist (rotate), span (stretch), taper, chord (scaleX). Setting the ref_axis_pos, the user can decide around with reference line the geometric variable will be applied, ref_axis_pos = 0.25 is the quarter of chord and was the default one. ref_axis_pos = 1 is around the trailing edge, ref_axis_pos = 0 is around leading edge.
Expected time until merged
a week
Type of change
Testing
Verified that the geometric variable have the correct behavior on simple cases (opt_chord.py, opt_twist.py)
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