#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
-~----------~----~----~----~------~----~------~--~---

Reply via email to