Skip to content

Conversation

@jebob
Copy link
Contributor

@jebob jebob commented May 19, 2025

This should fix #811? Sadly, I don't have working tests (see #840) or access to the solvers.

@pchtsp
Copy link
Collaborator

pchtsp commented May 20, 2025

Thanks. Can you add a unit test with the error you're trying to fix? that way we're sure if it actually solves it or not.

@jebob
Copy link
Contributor Author

jebob commented May 28, 2025

I've added a test, but I cannot verify that it triggers an error (as I don't have Gurobi locally). I can't trigger the Github pipelines myself, so I can't test the test with and without the fix.

@pchtsp
Copy link
Collaborator

pchtsp commented May 28, 2025

You can install gurobi locally by installing

pulp[public_py]

This will install the public python APIs, including gurobipy.

@pchtsp
Copy link
Collaborator

pchtsp commented May 28, 2025

And you need to format with black

@jebob
Copy link
Contributor Author

jebob commented May 30, 2025

You can install gurobi locally by installing

pulp[public_py]

This will install the public python APIs, including gurobipy.

Aha, this installs the GUROBI solver, whereas I've patched the GUROBI_CMD solver, which is why I am struggling to reproduce the bug locally. Nonetheless, I think the test is correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Solver fails if TMPDIR includes a space

2 participants