#14772: Remove CombinatorialClass from Permutations
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-5.12
      Component:  combinatorics      |   Resolution:
       Keywords:  days49             |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:  Mike Hansen, Darij
Report Upstream:  N/A                |  Grinberg, Jeff Ferreira
         Branch:                     |  Work issues:
   Dependencies:  #8386 #14519       |       Commit:
  #14808 #14143 #14015 #14516        |     Stopgaps:
  #14863                             |
-------------------------------------+-------------------------------------

Comment (by zabrocki):

 I'm glad that this is not intentional.  I was worried that this patch made
 more significant changes to the use of the {{{Permutation}}} class.

 {{{
 sage: Permutation([3,2,1])
 ---------------------------------------------------------------------------
 TypeError                                 Traceback (most recent call
 last)
 <ipython-input-1-b3e6779bea6e> in <module>()
 ----> 1 Permutation([Integer(3),Integer(2),Integer(1)])

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/misc/classcall_metaclass.so in
 sage.misc.classcall_metaclass.ClasscallMetaclass.__call__
 (sage/misc/classcall_metaclass.c:1224)()

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/combinat/permutation.pyc in __classcall_private__(cls, l,
 check_input)
     499
     500         # otherwise, it gets processed by CombinatorialObject's
 __init__.
 --> 501         return Permutations()(l, check_input=check_input)
     502
     503     def __init__(self, parent, l, check_input=True):

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/structure/parent.so in sage.structure.parent.Parent.__call__
 (sage/structure/parent.c:8154)()

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/structure/coerce_maps.so in
 sage.structure.coerce_maps.DefaultConvertMap_unique._call_with_args
 (sage/structure/coerce_maps.c:4278)()

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/structure/coerce_maps.so in
 sage.structure.coerce_maps.DefaultConvertMap_unique._call_with_args
 (sage/structure/coerce_maps.c:4089)()

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/categories/sets_cat.pyc in
 _element_constructor_from_element_class(self, *args, **keywords)
     337                 17
     338             """
 --> 339             return self.element_class(parent = self, *args,
 **keywords)
     340
     341         def is_parent_of(self, element):

 /Applications/sage-5.11.rc0/local/lib/python2.7/site-
 packages/sage/misc/classcall_metaclass.so in
 sage.misc.classcall_metaclass.ClasscallMetaclass.__call__
 (sage/misc/classcall_metaclass.c:1243)()

 TypeError: __init__() got multiple values for keyword argument 'parent'
 }}}

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