coin-or / Clp.old

This a mirror of the subversion repository on COIN-OR.
https://projects.coin-or.org/Clp
Other
36 stars 20 forks source link

[TRAC] Comparison of double with integer #22

Open qtothec opened 6 years ago

qtothec commented 6 years ago

Moved from TRAC as part of COIN fORgery: https://projects.coin-or.org/Clp/ticket/50

In stable/1.14, in the isBinary() method of OsiClpSolverInterface?.cpp around line 2718, a comparison of bounds with 1 and 0 is made without tolerances. But since bounds are double numbers this might create problems. I have an example where a lower bound with value 1e-15 results in the method to return false.

jjhforrest commented 5 years ago

Test should not be changed. Should be trapped earlier in code - so for input lp it would be. Need example.