#8977: Bug in QuadraticForm.rational_diagonal_form()
-------------------------------+--------------------------------------------
   Reporter:  gdrama           |       Owner:  justin                  
       Type:  defect           |      Status:  needs_work              
   Priority:  minor            |   Milestone:                          
  Component:  quadratic forms  |    Keywords:  rational_diagonal_form()
     Author:                   |    Upstream:  N/A                     
   Reviewer:                   |      Merged:                          
Work_issues:                   |  
-------------------------------+--------------------------------------------

Comment(by mstreng):

 Can anyone reproduce this? A seemingly unrelated doctest fails.
 {{{
 The following tests failed:

         sage -t -long devel/sage/sage/groups/matrix_gps/matrix_group.py #
 4 doctests failed
 }}}
 All 4 are of the following form:
 {{{
 File "/storage/marco/sage-4.6.1/devel/sage-
 main/sage/groups/matrix_gps/matrix_group.py", line 668:
     sage: G.random_element()
 Expected:
     [2 1 1 1]
     [1 0 2 1]
     [0 1 1 0]
     [1 0 0 1]
 Got:
     [0 1 1 0]
     [1 2 2 2]
     [1 1 1 0]
     [2 0 1 2]
 }}}
 I tried removing and reapplying the patch. All tests pass on an unpatched
 4.6.2.alpha3, and 4 tests of the form {{{G.random_element()}}} fail with
 the patch applied. I've had problems like this before with other tickets,
 so I won't change this ticket to needs_work (yet).

 Oops, it seems that the ticket still is needs_work. Was it not ready for
 review yet?

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8977#comment:7>
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.

Reply via email to