-
Notifications
You must be signed in to change notification settings - Fork 38
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
fix and extend tests for conic duals #82
Conversation
CC @ulfworsoe |
Looks good on phone. I could have sworn we sanity checked the signs on these |
Ok... I am probably misunderstanding something. What is your definition of Using the model from S = { x > 0 }, S* = { s > 0 }
Is this correct? On Tue, Aug 11, 2015 at 3:20 PM, Iain Dunning notifications@github.com
Ulf Worsøe |
@ulfworsoe, yes, NonNeg, NonPos, SOC, and SDP are self dual. |
Now I am just being stupid. Doesn't this test On Tue, Aug 11, 2015 at 4:12 PM, Miles Lubin notifications@github.com
Ulf Worsøe |
@ulfworsoe, yes, but that should be fixed in this PR |
Let me know if anyone finds something off here, otherwise I'll merge and tag so we can get to fixing the solvers. |
👍 |
fix and extend tests for conic duals
CC @madeleineudell @kaarthiksundar @IainNZ @joehuchette