coin-or / pulp

A python Linear Programming API
http://coin-or.github.io/pulp/
Other
2.1k stars 384 forks source link

`fixValue()` uses the wrong variables #642

Closed MLopez-Ibanez closed 1 year ago

MLopez-Ibanez commented 1 year ago

https://github.com/coin-or/pulp/blob/1366ad85fad1b5e8dda2b44cd143adeb206e2565/pulp/pulp.py#L653-L659

The variables self._lowbound_unfix and self._upbound_unfix are never used anywhere else. I would also argue that they are useless since when a variable is created, __init__ already sets self._lowbound_original, self._upbound_original