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

Hook block matrices into matrix(...) command #1755

Closed
robertwb opened this issue Jan 11, 2008 · 3 comments
Closed

Hook block matrices into matrix(...) command #1755

robertwb opened this issue Jan 11, 2008 · 3 comments

Comments

@robertwb
Copy link
Contributor

Add the ability to access the functionality of #1732 to the default matrix(...) constructor.

Component: linear algebra

Reviewer: Marc Mezzarobba

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

@robertwb robertwb added this to the sage-5.11 milestone Jan 11, 2008
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@black-stones
Copy link
Mannequin

black-stones mannequin commented May 13, 2018

comment:6

This is already implemented as block_matrix()

sage: A = random_matrix(ZZ, 2)
sage: B = random_matrix(ZZ, 2)
sage: C = random_matrix(ZZ, 2)
sage: D = random_matrix(ZZ, 2)
sage: block_matrix( [[A, B], [C, D]] )
[  1  -3| -1  -1]
[  0   0|  1   1]
[-------+-------]
[  0   0| -5   1]
[ -1   0|-10  31]

I don't think matrix() should create block matrices. It's possible that one would want a matrix of matrices rather than to merge the matrices together.

@black-stones black-stones mannequin removed this from the sage-6.4 milestone May 13, 2018
@black-stones black-stones mannequin added the s: needs review label May 13, 2018
@mezzarobba
Copy link
Member

Reviewer: Marc Mezzarobba

@embray
Copy link
Contributor

embray commented Feb 26, 2019

comment:8

Presuming these are all correctly reviewed as either duplicate, invalid, or wontfix.

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

6 participants