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

WIP: adding cplex annotations to be used for bender decomposition #125

Merged
merged 5 commits into from
Apr 24, 2017
Merged

WIP: adding cplex annotations to be used for bender decomposition #125

merged 5 commits into from
Apr 24, 2017

Conversation

IssamT
Copy link
Contributor

@IssamT IssamT commented Apr 14, 2017

We are implementing a new package BlockJuMP to define block decompositions for MIPs. It will also define oracles to the solve the subproblems and other useful directives. See an example here: https://realopt.github.io/BlockJuMP.jl/latest/basic.html.

Cplex is one of the solvers that is already supporting benders decomposition and we would like to use it with BlockJuMP. What do you think about it ?

@mlubin
Copy link
Member

mlubin commented Apr 14, 2017

This is going to require a longer discussion, but a few quick comments.

  1. Cool!
  2. You should be aware of StructJuMP which seemingly has a similar purpose to BlockJuMP. You're of course free to choose to make your own extension, but I would ask that you not use "JuMP" in the name of it given that none of the JuMP developers are involved and StructJuMP is an existing JuMP extension (with JuMP developers involved) having a similar purpose.
  3. We're unlikely to merge a PR that introduces optional dependencies on external packages or implements anything except wrappers for the CPLEX C API or the official MathProgBase interface. newlongannotation, for example, is fine but a dependency on BlockJuMP is not.

If you have the opportunity to attend, the JuMP developers meetup would be a good place to discuss this in depth.

@mlubin mlubin merged commit 35f0c91 into jump-dev:master Apr 24, 2017
blegat pushed a commit that referenced this pull request Apr 29, 2017
* added annotations to handle benders

* solver removed from CplexModel

* Removing cstrs decomposition (DW dec not supported)

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

Successfully merging this pull request may close these issues.

3 participants