Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#11114 closed enhancement (fixed)

is_diagonalizable() for matrices

Reported by: rbeezer Owned by: jason, was
Priority: minor Milestone: sage-4.7.1
Component: linear algebra Keywords:
Cc: Merged in: sage-4.7.1.alpha0
Authors: Rob Beezer Reviewers: Dan Drake
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Title says it all.

See comments in source about a minimal polynomial approach probably only being workable in a derived class with an efficient implementation of the minimal polynomial.

Attachments (1)

trac_11114-diagonalizable-matrices.patch (7.7 KB) - added by rbeezer 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by rbeezer

comment:1 Changed 9 years ago by rbeezer

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

comment:2 follow-up: Changed 9 years ago by ddrake

  • Reviewers set to Dan Drake
  • Status changed from needs_review to positive_review

Nice work. Passes doctests, applies to 4.7.alpha3. Positive review.

comment:3 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-4.7 to sage-4.7.1

comment:4 Changed 9 years ago by jdemeyer

  • Merged in set to sage-4.7.1.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:5 in reply to: ↑ 2 Changed 9 years ago by rbeezer

Replying to ddrake:

Nice work. Passes doctests, applies to 4.7.alpha3. Positive review.

Thanks for the review, Dan! (Just getting caught up on my own bookkeeping with various outstanding patches.)

Note: See TracTickets for help on using tickets.