#10794 closed enhancement (fixed)
QR decomposition for matrices over exact rings
Reported by: | rbeezer | Owned by: | jason, was |
---|---|---|---|
Priority: | major | Milestone: | sage-4.7 |
Component: | linear algebra | Keywords: | matrix, QR |
Cc: | Merged in: | sage-4.7.alpha5 | |
Authors: | Rob Beezer | Reviewers: | Simon Spicer |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Attachments (1)
Change History (7)
Changed 10 years ago by
comment:1 Changed 10 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
- Description modified (diff)
comment:3 follow-up: ↓ 4 Changed 10 years ago by
- Keywords matrix QR added
- Reviewers set to Simon Spicer
- Status changed from needs_review to positive_review
Hi Rob
Looks good. All doctests pass, all tests I cooked up passed, and the code looks good. I couldn't even find a typo.
Out of interest, over what fields other than QQbar will this method be called?
Simon
comment:4 in reply to: ↑ 3 Changed 10 years ago by
Replying to spice:
Out of interest, over what fields other than QQbar will this method be called?
Thanks, again, Simon. Isn't QQbar enough? ;-) Once somebody implements Fpbar (algebraic closure of finite fields of characteristic p) then we will have another.
Rob
comment:5 Changed 10 years ago by
- Merged in set to sage-4.7.alpha5
- Resolution set to fixed
- Status changed from positive_review to closed
comment:6 Changed 10 years ago by
Note: See
TracTickets for help on using
tickets.
fix patchbot comment