search

coin-or / pulp

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

handle PL bounds and include test #745

Closed rutger-van-beek-cqm closed 3 months ago

rutger-van-beek-cqm commented 3 months ago

Fixes #635.

Some software writes MPS-files including PL bounds, but without a value. When reading these lines the readMPSSetBounds raises an IndexError, when trying to read the value.

PL-bounds can be ignored. This PR implements a case in the readMPSSetBounds to ignore the PL-type bounds.

Next to that a test is added, to test whether a MPS-file containing a PL-bound can be read.

CLAassistant commented 3 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: rutger-van-beek-cqm
:x: Rutger van Beek

Rutger van Beek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

rutger-van-beek-cqm commented 3 months ago

I will make a new PR where the commit can be linked to my account.