#6942: jordan_form with transformation=true returns non-invertible
transformation
-------------------------------------------+--------------------------------
Reporter: syazdani | Owner: tbd
Type: defect | Status: positive_review
Priority: critical | Milestone: sage-4.3.2
Component: linear algebra | Keywords: jordan_form,
transformation
Author: Sebastian Pancratz | Upstream: N/A
Reviewer: Rob Beezer, Minh Van Nguyen | Merged:
Work_issues: |
-------------------------------------------+--------------------------------
Comment(by spancratz):
Dear Rob,
I am sorry that I am only looking at this again now. Of course, the
reviewer patch looks fine. Thanks again for reviewing this! About your
other points...
3) Yes, that is a good observation. I am almost sure that if one looked
closer at the linear algebra operations then this code could easily be
improved. There are currently two reasons why I won't look at this,
though. (a) The previous code was broken, so I think it's important to
first have something that "obviously" works, in the sense that it is
moderately easy to review since the code only uses "high-level"
operations. (b) I've got my first year interview this coming Monday.
4) This is completely up to you. The matrix looks quite intriguing and
more examples certainly couldn't hurt. Since this ticket will probably be
closed soon, if you decide to include this new example, feel free to let
me know (via email or sage-devel) and I'll review it right away.
Also, Minh, thank you for picking up the slack and completing the review
process!
Kind regards,
Sebastian
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6942#comment:11>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
