Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Mar 27, 2023

📚 Description

To replace sage_numerical_backends_coin, we need several improvements of the cvxpy+cylp backend:

Resolves #30644

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@mantepse
Copy link
Contributor

mantepse commented Apr 1, 2023

I'm not sure whether I am a good reviewer for that - I have no idea what I should actually check.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 1, 2023

It's waiting for the dependency #35369 anyway

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jun 5, 2023

Next steps:

  • Collect examples from doctests in sage.graphs or sage.combinat that take nontrivial time with GLPK
  • Test with the backend; should typically be faster; if not, check for performance losses in interfacing

@github-actions
Copy link

github-actions bot commented Jun 5, 2023

Documentation preview for this PR (built with commit 96651d2) is ready! 🎉

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade optional package cbc to 2.10.x, remove sage_numerical_backends_coin

3 participants