Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

allow to remove no constraints #34881

Closed
mantepse opened this issue Dec 28, 2022 · 7 comments
Closed

allow to remove no constraints #34881

mantepse opened this issue Dec 28, 2022 · 7 comments
Labels
c: linear programming p: major / 3 t: bug
Milestone
sage-9.8

Comments

@mantepse
Copy link
Contributor

Removing no constraint makes sage glpk currently crash, which we fix here.

CC: @mkoeppe

Component: linear programming

Author: Martin Rubey

Branch/Commit: b03cf22

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/34881
@mantepse mantepse added this to the sage-9.8 milestone Dec 28, 2022
@mantepse
Copy link
Contributor Author

Branch: u/mantepse/allow_to_remove_no_constraints

@mantepse
Copy link
Contributor Author

New commits:

b03cf22take care of the corner case for all backends in the frontend

@mantepse

This comment has been minimized.

Sign in to view
@mantepse
Copy link
Contributor Author

Commit: b03cf22

@mantepse
Copy link
Contributor Author

Author: Martin Rubey

@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 28, 2022

Reviewer: Matthias Koeppe

@vbraun
Copy link
Member

vbraun commented Jan 12, 2023

Changed branch from u/mantepse/allow_to_remove_no_constraints to b03cf22

@vbraun vbraun closed this as completed in 3570a9a Jan 12, 2023
@mantepse mantepse mentioned this issue Jan 29, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: linear programming p: major / 3 t: bug
Projects
None yet
Milestone
sage-9.8
Development

No branches or pull requests
3 participants
@vbraun @mkoeppe @mantepse