Opened 12 years ago

Closed 15 months ago

#1755 closed enhancement (invalid)

Hook block matrices into matrix(...) command

Reported by: robertwb Owned by: was
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: linear algebra Keywords:
Cc: Merged in:
Authors: Reviewers: Marc Mezzarobba
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:


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

Change History (8)

comment:1 Changed 11 years ago by malb

  • Type changed from defect to enhancement

comment:2 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:3 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:4 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:5 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:6 Changed 2 years ago by gh-black-stones

  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Report Upstream set to N/A
  • Status changed from new to needs_review

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.

comment:7 Changed 2 years ago by mmezzarobba

  • Reviewers set to Marc Mezzarobba
  • Status changed from needs_review to positive_review

comment:8 Changed 15 months ago by embray

  • Resolution set to invalid
  • Status changed from positive_review to closed

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

Note: See TracTickets for help on using tickets.