Skip to content
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

Lagrange multiplier graphs for Becca to look at #41

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rebeccamccabe
Copy link
Collaborator

@rebeccamccabe rebeccamccabe commented Mar 20, 2024

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.
Active Constraints
Lagranges

Copy link

codecov bot commented Mar 20, 2024

Codecov Report

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.

❗ 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 ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   78.69%   79.43%   +0.74%     
==========================================
  Files          37       37              
  Lines        1577     1634      +57     
==========================================
+ Hits         1241     1298      +57     
  Misses        336      336              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

% I'll change these later
for i=1:m
for j = 1:n
if rem(j,14)==1
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Outside loop: color_cell = { '.r', 'or', etc ... }
color = color_cell{ rem(j,14) }

@rebeccamccabe
Copy link
Collaborator Author

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.

@rebeccamccabe
Copy link
Collaborator Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants