Opened 5 years ago

Closed 5 years ago

#25504 closed defect (fixed)

Implement _an_element_ for matrix spaces

Reported by: Jeroen Demeyer Owned by:
Priority: major Milestone: sage-8.3
Component: linear algebra Keywords:
Cc: Vincent Delecroix Merged in:
Authors: Jeroen Demeyer Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 779dffb (Commits, GitHub, GitLab) Commit: 779dffbbb514290400d2b164b80ddb1a6c8ba04e
Dependencies: Stopgaps:

Status badges

Description

Currently, this is a bit too trivial:

sage: MatrixSpace(QQ, 3, 3).an_element()
[1 0 0]
[0 0 0]
[0 0 0]

Change History (12)

comment:1 Changed 5 years ago by Jeroen Demeyer

Branch: u/jdemeyer/implement__an_element__for_matrix_spaces

comment:2 Changed 5 years ago by git

Commit: ac7cf066bd3dfffb0cc5292f55ce5d9d36094a0a

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

ac7cf06Implement _an_element_ for matrix spaces

comment:3 Changed 5 years ago by Jeroen Demeyer

Status: newneeds_review

comment:4 Changed 5 years ago by Jeroen Demeyer

Status: needs_reviewneeds_work

See patchbot

comment:5 Changed 5 years ago by git

Commit: ac7cf066bd3dfffb0cc5292f55ce5d9d36094a0a2c09211e782ea84946e25636c3ae25049ca50425

Branch pushed to git repo; I updated commit sha1. New commits:

2c09211Doctest fixes

comment:6 Changed 5 years ago by Jeroen Demeyer

Status: needs_workneeds_review

comment:7 Changed 5 years ago by Travis Scrimshaw

If you could add a test M.density() for the sparse test to verify that it is indeed sparse (beyond the implicit test that you would kill your memory if such a sparse matrix was dense). Actually, that test makes me a bit nervous if at some point it does turn into a dense matrix. It would make it nearly impossible to debug because my computer would keep running out of memory. Perhaps it would be better to scale down the matrix and just do M.density() to show it is sparse.

comment:8 Changed 5 years ago by git

Commit: 2c09211e782ea84946e25636c3ae25049ca50425e09aed6e246816e232680ceba22096a756cf9ba8

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

e09aed6Implement _an_element_ for matrix spaces

comment:9 Changed 5 years ago by Jeroen Demeyer

I updated the branch.

comment:10 Changed 5 years ago by git

Commit: e09aed6e246816e232680ceba22096a756cf9ba8779dffbbb514290400d2b164b80ddb1a6c8ba04e

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

779dffbImplement _an_element_ for matrix spaces

comment:11 Changed 5 years ago by Travis Scrimshaw

Reviewers: Travis Scrimshaw
Status: needs_reviewpositive_review

Thank you. LGTM.

comment:12 Changed 5 years ago by Volker Braun

Branch: u/jdemeyer/implement__an_element__for_matrix_spaces779dffbbb514290400d2b164b80ddb1a6c8ba04e
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.