Open rebeccamccabe opened 5 months ago
Attention: Patch coverage is 98.52941%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 79.43%. Comparing base (
49bee41
) to head (fcf6f12
). Report is 2 commits behind head on main.:exclamation: Current head fcf6f12 differs from pull request most recent head 152fd8a. Consider uploading reports for the commit 152fd8a to get more accurate results
Files | Patch % | Lines |
---|---|---|
mdocean/optimization/gradient_optim.m | 75.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
for the most part, which constraints are active is consistent between the two ways of plotting, but the number on the x-axis is not consistent.
First count number of constraints and get confident on whether it's a reordering issue on the x-axis or some other issue in computation. If it's a reordering issue, look in the generator folder and see if there's a way that the constraints are getting reordered.
If a constraint is active for certain designs, then that constraint has a non-zero lagrange multiplier for certain other designs. The problem is that the designs that have active constraints are not necessarily the designs that have the non-zero lagrange multipliers.