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.

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

crash with MIP_Problem #19903

Closed
videlec opened this issue Jan 16, 2016 · 13 comments
Closed

crash with MIP_Problem #19903

videlec opened this issue Jan 16, 2016 · 13 comments

Comments

@videlec
Copy link
Contributor

videlec commented Jan 16, 2016

The following crashes Sage

sage: from sage.libs.ppl import Variable, Constraint_System, MIP_Problem
sage: x = Variable(0)
sage: y = Variable(1)
sage: cs = Constraint_System()
sage: cs.insert(x + y <= 2)
sage: MIP_Problem(2, cs, 0)
...
Unhandled SIGSEGV: A segmentation fault occurred in Sage.

If we replace 0 by x or y it also crashes... but with x-x or x+y or x+0it is fine!

This is just a problem of types.

CC: @vbraun

Component: linear programming

Author: Vincent Delecroix

Branch/Commit: f0c0346

Reviewer: Volker Braun, Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/19903

@videlec videlec added this to the sage-7.0 milestone Jan 16, 2016
@videlec
Copy link
Contributor Author

videlec commented Jan 16, 2016

New commits:

d22dbfaTrac 19903: fix MIP_Problem constructor

@videlec
Copy link
Contributor Author

videlec commented Jan 16, 2016

Branch: u/vdelecroix/19903

@videlec
Copy link
Contributor Author

videlec commented Jan 16, 2016

Commit: d22dbfa

@vbraun
Copy link
Member

vbraun commented Jan 16, 2016

comment:2

cdef attribute initialization should be done from __cinit__, not __init__

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 16, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

980f125Trac 19903: add a 'c'

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 16, 2016

Changed commit from d22dbfa to 980f125

@vbraun
Copy link
Member

vbraun commented Jan 17, 2016

comment:4

lgtm

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 3, 2016

comment:5

Patch no longer applies to current Sage, needs rebasing.

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 3, 2016

Reviewer: Volker Braun

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 3, 2016

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

f0c0346Trac 19903: fix MIP_Problem constructor

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 3, 2016

Changed commit from 980f125 to f0c0346

@videlec videlec modified the milestones: sage-7.0, sage-7.2 Apr 3, 2016
@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 4, 2016

Changed reviewer from Volker Braun to Volker Braun, Matthias Koeppe

@vbraun
Copy link
Member

vbraun commented Apr 5, 2016

Changed branch from u/vdelecroix/19903 to f0c0346

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

No branches or pull requests

3 participants