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

SystemError in cmp for matrix_integer_2x2 #14688

Closed
rharron mannequin opened this issue Jun 5, 2013 · 4 comments
Closed

SystemError in cmp for matrix_integer_2x2 #14688

rharron mannequin opened this issue Jun 5, 2013 · 4 comments
Assignees
@jasongrout @williamstein
Labels
c: linear algebra p: major / 3 t: bug
Milestone
sage-5.12

Comments

@rharron
Copy link
Mannequin

rharron mannequin commented Jun 5, 2013

The following throws a SystemError:

sage: from sage.matrix.matrix_integer_2x2 import MatrixSpace_ZZ_2x2
sage: M2ZSpace = MatrixSpace_ZZ_2x2()
sage: A = M2ZSpace([-5, -3, 7, 4])
sage: B = M2ZSpace([1,0,-2,1])
sage: A < B
---------------------------------------------------------------------------
SystemError                               Traceback (most recent call last)
<ipython-input-60-e9066daab57d> in <module>()
----> 1 A < B

SystemError: error return without exception set

(I think this was probably found and fixed by David Roe, but he hasn't posted anything on trac, so I'm doing that).

Component: linear algebra

Keywords: SystemError, matrix_integer_2x2, cmp

Author: Robert Harron

Reviewer: Jeroen Demeyer

Merged: sage-5.12.beta4

Issue created by migration from https://trac.sagemath.org/ticket/14688
@rharron rharron mannequin added this to the sage-5.11 milestone Jun 5, 2013
@rharron rharron mannequin added c: linear algebra labels Jun 5, 2013
@rharron rharron mannequin assigned jasongrout and williamstein Jun 5, 2013
@rharron
Copy link
Mannequin Author

rharron mannequin commented Jun 5, 2013

Author: Robert Harron

@rharron
Copy link
Mannequin Author

rharron mannequin commented Jun 5, 2013

comment:1

Attachment: trac_14688_fix_compare_in_matrix_integer_2x2.patch.gz

@rharron rharron mannequin added the s: needs review label Jun 5, 2013
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

Merged: sage-5.12.beta4

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

@jasongrout jasongrout

@williamstein williamstein

Labels
c: linear algebra p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.12
Development

No branches or pull requests
3 participants
@jasongrout @williamstein @jdemeyer