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

proposed revision of conic interface #38

Merged
merged 3 commits into from
Sep 25, 2014
Merged

proposed revision of conic interface #38

merged 3 commits into from
Sep 25, 2014

Conversation

mlubin
Copy link
Member

@mlubin mlubin commented Aug 21, 2014

As discussed in #36, here's a proposed revision to the conic interface which allows the user to specify conic restrictions on both variables and affine functions of variables. Solver interfaces will be responsible for massaging the problem into the format required by each solver, but this interface makes it easier on the modeling side by avoiding the need for slacks and identity transformations.

The primal is:

min c'x
s.t. Ax - b in K_1
     x in K_2

and the dual is:

max b'y
s.t.   c - A'y in K_2^*
       y in K_1^*

The choice of signs for the affine constraints is arbitrary but results in a dual that's reasonably aesthetically pleasing to me (negative signs in the objective seem ugly).

Here's a preview of the docs: http://mathprogbasejl.readthedocs.org/en/conic2/conic.html

@madeleineudell @karanveerm @IainNZ @joehuchette

@IainNZ
Copy link
Member

IainNZ commented Sep 2, 2014

@madeleineudell @karanveerm you cool with this?

@madeleineudell
Copy link

Hi! Sorry for the long delay. Flipping the sign on the first constraint in
the primal problem to

min c'x
s.t.  b - Ax in K_1
       x in K_2

makes it look better to me; then the constraint is equivalent to $$Ax + s =
b$$ where the slack variable $$s \in K_1$$; and it's standard to have a
negative sign in the dual objective. But flipping the sign doesn't seem
particularly important. Other than that, the proposed interface looks
excellent.

On Tue, Sep 2, 2014 at 3:01 PM, Iain Dunning notifications@github.com
wrote:

@madeleineudell https://github.com/madeleineudell @karanveerm
https://github.com/karanveerm you cool with this?


Reply to this email directly or view it on GitHub
#38 (comment)
.

Madeleine Udell
PhD Candidate in Computational and Mathematical Engineering
Stanford University
www.stanford.edu/~udell

@mlubin
Copy link
Member Author

mlubin commented Sep 24, 2014

Cool! I updated the signs as you suggested. I'll merge this soon.

mlubin added a commit that referenced this pull request Sep 25, 2014
proposed revision of conic interface
@mlubin mlubin merged commit 6b23322 into master Sep 25, 2014
@mlubin mlubin deleted the conic2 branch September 25, 2014 21:42
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.

None yet

3 participants