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: |
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)
Change History (8)
Changed 11 years ago by
comment:1 Changed 11 years ago by
- Status changed from new to needs_review
Changed 11 years ago by
comment:2 Changed 11 years ago by
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
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
- 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
comment:6 Changed 11 years ago by
- Merged in set to sage-4.6.alpha1
- Resolution set to fixed
- Status changed from positive_review to closed
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