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

Elements of GL(n,R) should coerce properly to matrices #3400

Closed
rlmill mannequin opened this issue Jun 11, 2008 · 5 comments
Closed

Elements of GL(n,R) should coerce properly to matrices #3400

rlmill mannequin opened this issue Jun 11, 2008 · 5 comments
Assignees
@mwhansen
Labels
c: coercion p: major / 3 t: bug
Milestone
sage-4.3.1

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jun 11, 2008

For example:

sage: M = Matrix(GF(2), [[1,1,1,1]])
sage: G = GL(4,2)
sage: N = G.0
sage: M
[1 1 1 1]
sage: N

[1 1 0 0]
[0 1 0 0]
[0 0 1 0]
[0 0 0 1]
sage: M*N
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)

/Users/rlmill/sage-3.0.2/<ipython console> in <module>()

/Users/rlmill/sage-3.0.2/element.pyx in sage.structure.element.Matrix.__mul__ (sage/structure/element.c:11352)()

/Users/rlmill/sage-3.0.2/coerce.pyx in sage.structure.coerce.CoercionModel_cache_maps.bin_op_c (sage/structure/coerce.c:5301)()

TypeError: unsupported operand parent(s) for '*': 'Full MatrixSpace of 1 by 4 dense matrices over Finite Field of size 2' and 'General Linear Group of degree 4 over Finite Field of size 2'
sage: M*N.matrix()
[1 0 1 1]

CC: @aghitza

Component: coercion

Author: Robert Bradshaw

Reviewer: Rob Beezer

Merged: sage-4.3.1.rc1

Issue created by migration from https://trac.sagemath.org/ticket/3400
@rlmill rlmill mannequin added this to the sage-4.3.1 milestone Jun 11, 2008
@rlmill rlmill mannequin added c: coercion labels Jun 11, 2008
@rlmill rlmill mannequin assigned robertwb Jun 11, 2008
@mwhansen mwhansen assigned mwhansen and unassigned robertwb Sep 19, 2008
@robertwb
Copy link
Contributor

comment:3

Attachment: 3400-matrix-group-action.patch.gz

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Jan 18, 2010

comment:4

Passes all tests and works as advertised. Positive review.

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Jan 18, 2010

Reviewer: Rob Beezer

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Jan 18, 2010

Author: Robert Bradshaw

@rlmill
Copy link
Mannequin Author

rlmill mannequin commented Jan 18, 2010

Merged: sage-4.3.1.rc1

@rlmill rlmill mannequin removed the s: positive review label Jan 18, 2010
@rlmill rlmill mannequin closed this as completed Jan 18, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwhansen mwhansen

Labels
c: coercion p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.3.1
Development

No branches or pull requests
2 participants
@mwhansen @robertwb