#10335: Add domains for permutation groups
-----------------------------+----------------------------------------------
   Reporter:  mhansen        |          Owner:  joyner     
       Type:  enhancement    |         Status:  needs_work 
   Priority:  major          |      Milestone:  sage-5.0   
  Component:  group theory   |       Keywords:             
Work_issues:                 |       Upstream:  N/A        
   Reviewer:  Robert Miller  |         Author:  Mike Hansen
     Merged:                 |   Dependencies:             
-----------------------------+----------------------------------------------
Changes (by rbeezer):

  * status:  needs_review => needs_work


Comment:

 I ran  {{{make ptestlong}}} and got quite a few failures.  I even tried it
 twice - not sure what is up.

   *  Many were in  sage/doc
   *  Many were simply the new version of {{{_repr_}}} for the symmetric
 group, and its subgroups
   *  The {{{latex()}}} method is putting a comma between the cycles within
 each generator.  Is this desired?
 {{{
 return ",".join(["(" + ",".join([latex(x) for x in cycle])+")" for cycle
 in self.cycle_tuples()])
 }}}
     At a minimum it is causing doctest failures.
   * 73 failures in sage/rings/number_field/galois_group.py, likely all the
 same root cause

 Saw these in the patch itself:

   * a stray "deprecated_function_alias" in a docstring
   * {{{``selfs``}}} in a docstring

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