Make XpressSolver struct mutable for Alpine compatibility.#37
Make XpressSolver struct mutable for Alpine compatibility.#37joaquimg merged 1 commit intojump-dev:masterfrom hellemo:master
Conversation
|
I don´t see any specific reason. If cplex and gurobi are mutable, then xpress should also be, did you check them? |
|
Modifying internal fields of the solver structs is not really what was intended. The |
|
Thanks, I guess the RawParameter way is cleaner, but in the short/immediate term perhaps we could do like Cplex and Gurobi and make it mutable? https://github.com/JuliaOpt/CPLEX.jl/search?q=AbstractMathProgSolver&unscoped_q=AbstractMathProgSolver |
|
Let's make |
|
That's great, but Alpine still uses MPB. Maybe I'm confused, but that's why I propose to do like Cplex and Gurobi for now, and do it the proper way with MOI later. |
|
I don't have an objection to the proposed change to the MPB interface. |
I''m working on support for Xpress as mip solver for Alpine, and this change seems to be necessary given the way Alpine sets the settings for the subsolvers. Any reason to keep it the way it is?