bootphon / pygamma-agreement

Gamma Agreement in Python
MIT License
40 stars 8 forks source link

Change optimizer to MIP #7

Closed hadware closed 2 years ago

hadware commented 3 years ago

From this issue it seems that the solver we're currently using is not optimal (nor does it give correct results). Maybe we should consider switching to https://docs.python-mip.com/en/latest/quickstart.html .

hadware commented 3 years ago

Another possibility is to directly use CVXOPT (a dependendy of cvxpy) http://cvxopt.org/userguide/modeling.html

hadware commented 3 years ago

The MIP implementation works, but is way to slow for now, cf https://github.com/coin-or/python-mip/issues/146

ghost commented 2 years ago

python-mip being unreliable for solving successive mixed integer problems in its current state due to definition time, we chose to keep using CVXPY, this time with the CBC solver (the one used by python-mip) to address precision problems.