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

hanke -- disturbing matrix constructor issue involving rows versus columns #61

Closed
williamstein opened this issue Sep 14, 2006 · 3 comments

Comments

@williamstein
Copy link
Contributor

Hi William,

I just found a disturbing trait about Matrix constructions, and was
wondering if you could include a rows/columns flag to address it.

When a matrix is constructed from tuples, it assumes that you want to
use these numbers as rows, regardless of whether the tuples are
appropriately sized. Since there is no way of deciding which is meant
for square matrices, it seems reasonable to add an extra (mandatory)
flag to the constructor for a list of tuples to ask which is meant.

I hope your semster is going well. See you,

					-Jon (Hanke)
--------------------------------------------------------------------
 
sage: M2 = MatrixSpace(ZZ,2,4)(range(8)); M2
[0 1 2 3]
[4 5 6 7]
 
sage: M2.columns()
 [(0, 4), (1, 5), (2, 6), (3, 7)]
 
sage: MatrixSpace(ZZ,2,4)(M2.columns())
[0 4 1 5]
[2 6 3 7]
 
sage: M2 == MatrixSpace(ZZ,2,4)(M2.columns())
 False
 
sage: M2 == MatrixSpace(ZZ,2,4)(M2.rows())
 True

Component: basic arithmetic

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

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Aug 23, 2007

comment:1

This is still a problem with Sage 2.8.2. The 2.8.3 release might cut it a little close because there are still a large number of tickets left (to be fixed in roughly 1 day).

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-2.9 milestone Aug 23, 2007
@mwhansen mwhansen self-assigned this Sep 21, 2007
@williamstein williamstein modified the milestones: sage-2.9.1, sage-2.8.12 Nov 3, 2007
@williamstein
Copy link
Contributor Author

Attachment: 61.patch.gz

@williamstein
Copy link
Contributor Author

comment:5

Attachment: trac61-modsym.patch.gz

tobiasdiez pushed a commit to tobiasdiez/sage that referenced this issue Feb 22, 2024
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

2 participants