#14772: Remove CombinatorialClass from Permutations
------------------------------------+------------------------------------
       Reporter:  tscrim            |         Owner:  sage-combinat
           Type:  enhancement       |        Status:  needs_review
       Priority:  major             |     Milestone:  sage-5.11
      Component:  combinatorics     |    Resolution:
       Keywords:  days49            |     Merged in:
        Authors:  Travis Scrimshaw  |     Reviewers:
Report Upstream:  N/A               |   Work issues:
         Branch:                    |  Dependencies:  #8386 #14519 #14808
       Stopgaps:                    |
------------------------------------+------------------------------------

Comment (by darij):

 I've started the review -- but I'm stuck at {{{PermutationOptions}}} now.
 You have these doctests:

 {{{
     EXAMPLES::

         sage: p213 = Permutation([2,1,3])
         sage: p312 = Permutation([3,1,2])
         sage: PermutationOptions(mult='l2r', display='list')
         sage: po = PermutationOptions()
         sage: po['display']
         'list'
         sage: p213
         [2, 1, 3]
         sage: PermutationOptions(display='cycle')
         sage: p213
         (1,2)
         sage: PermutationOptions(display='singleton')
         sage: p213
         (1,2)(3)
         sage: PermutationOptions(display='list')
 }}}

 But when I try to run them ingame, they don't behave that nicely:

 {{{
 sage:         sage: p213 = Permutation([2,1,3])
 sage:         sage: p312 = Permutation([3,1,2])
 sage:         sage: PermutationOptions(mult='l2r', display='list')
 sage:         sage: po = PermutationOptions()
 Current options for permutations
   - display:        list
   - generator_name: s
   - latex:          list
   - mult:           l2r
 sage:         sage: po['display']
 ---------------------------------------------------------------------------
 TypeError                                 Traceback (most recent call
 last)
 <ipython-input-49-b0a2d723e60c> in <module>()
 ----> 1 po['display']

 TypeError: 'NoneType' object has no attribute '__getitem__'
 }}}

 I have a feeling that they are not recognized as doctests due to the weird
 place they're in... And I don't know what to do about this.

--
Ticket URL: <http://trac.sagemath.org/ticket/14772#comment:22>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to