NFC: Remove CBC from our installation documentation#1303
NFC: Remove CBC from our installation documentation#1303blnicho merged 1 commit intoPyomo:masterfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1303 +/- ##
==========================================
+ Coverage 68.61% 69.38% +0.76%
==========================================
Files 534 534
Lines 80977 81026 +49
==========================================
+ Hits 55566 56216 +650
+ Misses 25411 24810 -601
Continue to review full report at Codecov.
|
|
Is this something we plan to add back eventually, or is the idea to remove it permanently? |
|
My intention was to remove it permanently. It only appeared in the installation docs to provide a very brief note on a few solvers that were easy to install with Anaconda. I think our users would certainly benefit from a more detailed/comprehensive guide on installing various solvers however this type of documentation would be time consuming to develop and difficult to keep up-to-date. |
3 participants
Fixes #1302
Summary/Motivation:
This removes CBC from the list of solvers that we mention can be installed using conda in our installation documentation. @CatChenal pointed out that the conda package only works on Linux and OSX and @mrmundt pointed out that the newer version of CBC causes test failures (#1295). Rather than update the documentation with a bunch of caveats for a particular solver, I think the better approach is to remove it completely for now. We should probably add a section to the documentation that gives more guidance on installing different solvers but that is outside the scope of this PR.
Changes proposed in this PR:
Legal Acknowledgement
By contributing to this software project, I agree to the following terms and conditions for my contribution: