#13196: GL(n, GF(q)).random_element() is way too slow for what it does
--------------------------------------------------------------+-------------
       Reporter:  Bouillaguet                                 |         Owner:  
joyner         
           Type:  enhancement                                 |        Status:  
positive_review
       Priority:  trivial                                     |     Milestone:  
sage-5.2       
      Component:  group theory                                |    Resolution:  
               
       Keywords:  matrix group, finite field, random element  |   Work issues:  
               
Report Upstream:  N/A                                         |     Reviewers:  
               
        Authors:  Charles Bouillaguet                         |     Merged in:  
               
   Dependencies:                                              |      Stopgaps:  
               
--------------------------------------------------------------+-------------
Changes (by dimpase):

  * status:  needs_review => positive_review


Comment:

 Replying to [comment:11 Bouillaguet]:
 > Patch updated. The bug was that my function returned a matrix, instead
 of returning an element of the group (basically, it returned g.matrix()
 instead of returning g). Fix: coerce the result into the group again. It
 takes a bit of time, and there is likely a workaround, but this is better
 than nothing.

 OK!

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