Just a suggestion.... <br><br>If COIN intends to actually support other solvers (besides CLP/CBC), they should probably have one machine with access to all supported solvers and some kind of script to compile and run a unit test against all solvers - so that developers can check this before a commit.<br>
<br>I'd much rather see FULL support for 1,2 or 3 solvers than partial support for a bunch where most things don't work. I would think that each vendor would "donate" a few hours a month to making sure COIN/OSI/etc worked correctly with their solvers. If not, COIN should probably just drop support of that vendor. Doesn't make much sense to have partial support.<br>
<br><br><br><br><div class="gmail_quote"><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">This won't compile at all now when using #ifdef COIN_HAS_CPX<br>
<br><br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp: In constructor 'LAP::CglLandPSimplex::CglLandPSimplex(const OsiSolverInterface&, const CglLandP::CachedData&, const CglLandP::Parameters&, const LAP::Validator&)':<br>
../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:212: error: 'else' without a previous 'if'<br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: 'OsiXprSolverInterface' was not declared in this scope<br>
../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: 'cpxSi' was not declared in this scope<br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: expected type-specifier before 'OsiXprSolverInterface'<br>
../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: expected `>' before 'OsiXprSolverInterface'<br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: expected `(' before 'OsiXprSolverInterface'<br>
../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: expected primary-expression before '>' token<br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:213: error: expected `)' before ';' token<br>
../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp: At global scope:<br>../../../../Cgl/src/CglLandP/CglLandPSimplex.cpp:28: warning: 'void MyAssertFunc(bool, const std::string&, const std::string&, unsig<div><div>
</div><div class="h5"><br><br>
<br><div class="gmail_quote"><br><br></div></div></div></blockquote></div><br>