Opened 12 years ago

Closed 12 years ago

#10737 closed enhancement (fixed)

Extended echelon form of a matrix

Reported by: Rob Beezer Owned by: jason, was
Priority: minor Milestone: sage-4.7
Component: linear algebra Keywords:
Cc: Merged in: sage-4.7.alpha4
Authors: Rob Beezer Reviewers: John Palmieri
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by John Palmieri)

A convenience for teaching linear algebra. Doctests (at least) depend on #10543, which was discovered in the course of building this.

Depends: #10543


Apply trac_10737-extended-echelon-form.patch and trac_10737-referee-doc.patch.

Attachments (2)

trac_10737-extended-echelon-form.patch (9.1 KB) - added by Rob Beezer 12 years ago.
trac_10737-referee-doc.patch (980 bytes) - added by John Palmieri 12 years ago.
apply on top of other patch

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by Rob Beezer

comment:1 Changed 12 years ago by Rob Beezer

Authors: Rob Beezer
Status: newneeds_review

Changed 12 years ago by John Palmieri

apply on top of other patch

comment:2 Changed 12 years ago by John Palmieri

Description: modified (diff)
Reviewers: John Palmieri

The main patch looks good to me: positive review. All tests pass, and I also tested on things like 0x0 matrices. I've added a small patch rewording one sentence in the documentation; this requires review.

comment:3 in reply to:  2 Changed 12 years ago by Rob Beezer

Replying to jhpalmieri:

The main patch looks good to me: positive review. All tests pass, and I also tested on things like 0x0 matrices. I've added a small patch rewording one sentence in the documentation; this requires review.

Rewording looks great - thanks.

comment:4 Changed 12 years ago by John Palmieri

Status: needs_reviewpositive_review

comment:5 Changed 12 years ago by Jeroen Demeyer

Merged in: sage-4.7.alpha4
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.