Opened 11 years ago

Closed 11 years ago

#9802 closed enhancement (fixed)

Add random diagonalizable matrix to matrix/constructor.py

Reported by: bwonderly Owned by: jason, was
Priority: major Milestone: sage-4.6
Component: linear algebra Keywords:
Cc: rbeezer, wdj Merged in: sage-4.6.alpha1
Authors: Billy Wonderly Reviewers: David Joyner
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

This depends on #9720, so first apply the v3 patch from there. This method generates random diagonalizable matrices whose eigenvectors, if computed by hand, have only integer values. This routine is designed as educational tool, generating exam and homework problems, and problem generating interacts.

Attachments (2)

trac_9802-random-diagonalizable-matrix.patch (11.4 KB) - added by bwonderly 11 years ago.
trac_9802-random-diagonalizable-matrix-v2.patch (13.9 KB) - added by bwonderly 11 years ago.
revised to fit generalization of random_matrix constructor. Apply v4 from #9720 and v2 from #9803 and go from there. This patch is independent from #9754, but will be rebased as soon as either one gets a positive review

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by bwonderly

comment:1 Changed 11 years ago by bwonderly

  • Status changed from new to needs_review

Changed 11 years ago by bwonderly

revised to fit generalization of random_matrix constructor. Apply v4 from #9720 and v2 from #9803 and go from there. This patch is independent from #9754, but will be rebased as soon as either one gets a positive review

comment:2 Changed 11 years ago by bwonderly

The v2 patch is independent of #9754. There are revisions to the documentation of the random_diagonalizable_matrix routine, as well as to the random_matrix routine. The code is revised to fit with the generalization of the random_matrix constructor. First apply v4 from #9720, and then v2 from #9803 and go from there.

comment:3 Changed 11 years ago by wdj

This (with the other patches, as indicated above) applied fine to 4.5.1 and passed sage -testall.

Positive review, as far as I am concerned (and will be useful for me teaching linear algebra later in the semester!). Perhaps Mike Hansen should have the final say?

Thanks Rob and bwonderly!

comment:4 Changed 11 years ago by rbeezer

  • Reviewers set to David Joyner
  • Status changed from needs_review to positive_review

Mike Hansen looked in on #9803, so I've marked this as ready to go based on comments above. Thanks, David.

comment:5 Changed 11 years ago by bwonderly

Release Manager

#9720, #9803, #9802, #9754 is each dependent on the predecessor, merge in this order.

comment:6 Changed 11 years ago by mpatel

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