#10515: Removed bug in MatrixGroup.list()
-------------------------------+--------------------------------------------
   Reporter:  stumpc5          |       Owner:  joyner      
       Type:  defect           |      Status:  new         
   Priority:  major            |   Milestone:              
  Component:  group theory     |    Keywords:  matrix group
     Author:  Christian Stump  |    Upstream:  N/A         
   Reviewer:                   |      Merged:              
Work_issues:                   |  
-------------------------------+--------------------------------------------
 the list method for `MatrixGroup` worked only for ZZ and GF(p), but not
 for QQ. This one line patch should do the job.

 We had:
 {{{

 sage: gens = [matrix(ZZ,2,[1,0,0,1])]
 sage: MatrixGroup(gens).list()
 [[1 0]
 [0 1]]

 sage: gens = [matrix(GF(5),2,[1,0,0,1])]
 sage: MatrixGroup(gens).list()
 [[1 0]
 [0 1]]

 sage: gens = [matrix(QQ,2,[1,0,0,1])]
 sage: MatrixGroup(gens).list()
 ...
 AttributeError: 'RationalField_with_category' object has no attribute
 'multiplicative_generator'
 }}}

 Now, the third works as well:

 {{{
 sage: gens = [matrix(QQ,2,[1,0,0,1])]
 sage: MatrixGroup(gens).list()
 [[1 0]
 [0 1]]
 }}}

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