Adds CONOPT to solvers documentation#4193
Conversation
|
See this checklist: https://jump.dev/JuMP.jl/stable/developers/checklists/#Adding-a-new-solver-to-the-documentation You should also edit the installation table.
I'm going to discuss this privately with the core contributors. Let's proceed with it in
I think we'd need more than an intention. We'd probably also reserve the right to deprecate the package and archive it if GAMS stopped maintaining it. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4193 +/- ##
=======================================
Coverage 99.93% 99.93%
=======================================
Files 42 42
Lines 6278 6302 +24
=======================================
+ Hits 6274 6298 +24
Misses 4 4 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
We would be happy for a maintenance agreement to be put in place. Then we can include conditions about the expected maintenance. Setting out a maintenance agreement might serve well for other new and existing solvers in JuMP-dev as well. |
|
Regarding
A PR has been created for this JuliaPackaging/Yggdrasil#14008 You should add the checklist to https://jump.dev/JuMP.jl/stable/packages/solvers/, or at least a link. Its current location is not well connected to the "Adding new solvers" section. |
Our conclusion is no, and that we'd prefer to de-emphasise the distinction between packages in jump-dev and those outside it. I've opened a PR to update the docs: #4195. We can discuss more there. From a user-perspective, they just do We have a policy for this: https://jump.dev/pages/governance/#transferring-repositories-to-jump-dev The key part is
Commercial wrappers like Gurobi.jl and KNITRO.jl exist in jump-dev as a historical accident; they were originally written by the JuMP community, and we have maintained them (to various levels of effort) since. We would like more of the commercial solvers to move out of jump-dev and into the maintenance hands of others. One in-progress example is Xpress: https://github.com/fico-xpress/XpressAPI.jl. |
odow
left a comment
There was a problem hiding this comment.
This is blocked by GAMS-dev/CONOPT.jl#11. CONOPT is not yet a registered package.
This is reasonable. The changes in #4195 are very good, since it puts all solvers on the same level. Previously, the JuMP-dev solvers looked more official than the contributed solvers. |
|
CONOPT.jl is now a registered package JuliaRegistries/General#160410 |
Adding the CONOPT.jl package to the solvers documentation.
We would like to discuss the possibility of bringing CONOPT.jl under the JuMP-dev organisation, similar to KNITRO.jl. The intention would be for us to continue the maintenance of CONOPT.jl as part of the JuMP community. We would include the line "Contact GAMS Software if you encounter any problem with this interface or the solver." in the documentation, as is the case for KNITRO.jl.
Basic
MOI.Test.runtests. Some test excludes arepermissible, but the reason for skipping a particular test should be
documented.
use the solver with JuMP
Documentation
docs/src/installation.mdOptional
docs/packages.toml