#5837: [with patch, needs review] bug in rational_diagonal_form() from
QuadraticForm class
-----------------------------+----------------------------------------------
Reporter: LBerlioz | Owner: LBerlioz
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: quadratic forms | Keywords: QuadraticForm diagonal
-----------------------------+----------------------------------------------
Changes (by mabshoff):
* cc: tornaria (added)
* priority: minor => major
* milestone: sage-3.4.1 => sage-3.4.2
Comment:
Hi Luis,
at this stage only blockers will be fixed in 3.4.1, so this ticket is
getting bounced to 3.4.2. Since this bug produces mathematically incorrect
answers this is also certainly not minor ;)
You should also attach a patch with a doctest that demonstrates that the
problem has been fixed. Let me know if you have any questions.
Gonzalo: If you find the time can you review this?
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5837#comment:3>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---