Opened 10 years ago

Closed 10 years ago

#10737 closed enhancement (fixed)

Extended echelon form of a matrix

Reported by: rbeezer 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 jhpalmieri)

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 rbeezer 10 years ago.
trac_10737-referee-doc.patch (980 bytes) - added by jhpalmieri 10 years ago.
apply on top of other patch

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by rbeezer

comment:1 Changed 10 years ago by rbeezer

  • Authors set to Rob Beezer
  • Status changed from new to needs_review

Changed 10 years ago by jhpalmieri

apply on top of other patch

comment:2 follow-up: Changed 10 years ago by jhpalmieri

  • Description modified (diff)
  • Reviewers set to 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 10 years ago by rbeezer

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 10 years ago by jhpalmieri

  • Status changed from needs_review to positive_review

comment:5 Changed 10 years ago by jdemeyer

  • Merged in set to sage-4.7.alpha4
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.