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

Dualizing ParametricOptInterface #58

Open
matbesancon opened this issue Sep 9, 2019 · 4 comments
Open

Dualizing ParametricOptInterface #58

matbesancon opened this issue Sep 9, 2019 · 4 comments

Comments

@matbesancon
Copy link
Contributor

A model created with ParameterJuMP does not currently suppport dualization, the parameters are lost in the way.
Given the form given in the manual, a parameter p such that one constraint is of the form A_i x + b_i + p in C_i ends up with an additive coefficient in the objective of the dual: y_i (b_i + p).

My intuition is that things would be smoother with ParameterJuMP.jl implemented at the MOI level and with the sets being for example MOI.EqualTo{ParamAffineExpression}, but I recognize it puts a burden on maintenance efforts.

As of now, I haven't figured out a place where the parameters appear in types of the JuMP model or constraints so it might be tricky without modifying ParameterJuMP.jl

@joaquimg
Copy link
Member

The current plan is that we can connect with ParametricOptInterface, once that is stable enough.

@joaquimg joaquimg changed the title Dualizing ParameterJuMP Dualizing ~~ParameterJuMP~~ Apr 19, 2022
@joaquimg joaquimg changed the title Dualizing ~~ParameterJuMP~~ Dualizing ParameterJuMP Apr 19, 2022
@joaquimg joaquimg changed the title Dualizing ParameterJuMP Dualizing ParametricOptInterface Sep 29, 2022
@dsigler1234
Copy link

Hello,

I was wondering if Dualization.jl supports ParametricOptInterface parameters for LPs at this point or if this is still work in progress?

@guilhermebodin
Copy link
Collaborator

Hi @dsigler1234 I think it still does not work but we haven`t recently tested it. I might look at it later this week but would not count on it working.

@dsigler1234
Copy link

@guilhermebodin thanks for your quick reply. Would a pull request with this capability for LPs be of interest?

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

No branches or pull requests

4 participants