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

Copy is broken on 2x2 integer matrix (mutability is not set) #8294

Closed
hivert opened this issue Feb 17, 2010 · 5 comments
Closed

Copy is broken on 2x2 integer matrix (mutability is not set) #8294

hivert opened this issue Feb 17, 2010 · 5 comments

Comments

@hivert
Copy link

hivert commented Feb 17, 2010

sage: M = sage.matrix.matrix_integer_2x2.MatrixSpace_ZZ_2x2()
sage: mat = M([3,4,5,6])
sage: mat.is_mutable()
True
sage: mat = copy(mat)
sage: mat.is_mutable()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)

/home/hivert/<ipython console> in <module>()

/usr/local/sage2/local/lib/python2.6/site-packages/sage/matrix/matrix0.so in sage.matrix.matrix0.Matrix.is_mutable (sage/matrix/matrix0.c:3928)()

AttributeError: 'NoneType' object has no attribute 'is_mutable'

Component: linear algebra

Keywords: Matrix 2x2, mutability, copy

Author: Florent Hivert

Reviewer: Martin Raum

Merged: sage-4.3.4.alpha0

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

@hivert hivert added this to the sage-4.3.4 milestone Feb 17, 2010
@hivert
Copy link
Author

hivert commented Feb 17, 2010

comment:1

Attachment: trac_8294-matrix_2x2_copy_mutability_fix-fh.patch.gz

Should be ready for review.

Florent

@hivert hivert self-assigned this Feb 17, 2010
@sagetrac-mraum
Copy link
Mannequin

sagetrac-mraum mannequin commented Feb 20, 2010

Reviewer: Martin Raum

@sagetrac-mraum
Copy link
Mannequin

sagetrac-mraum mannequin commented Feb 20, 2010

comment:3

This applies cleanly and all tests pass. Positive review as is.
#8276 will follow soon.

@hivert
Copy link
Author

hivert commented Feb 20, 2010

comment:4

Replying to @sagetrac-mraum:

This applies cleanly and all tests pass. Positive review as is.
#8276 will follow soon.

Thanks a lot for this quick help !

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Mar 3, 2010

Merged: sage-4.3.4.alpha0

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

1 participant