#19513: remove expired deprecations in sage/combinat
-------------------------------------+-------------------------------------
       Reporter:  zabrocki           |        Owner:
           Type:  enhancement        |       Status:  new
       Priority:  major              |    Milestone:  sage-6.10
      Component:  combinatorics      |   Resolution:
       Keywords:  deprecate          |    Merged in:
        Authors:  Mike Zabrocki      |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  public/ticket/19513                |  2b6c11ae6423f0e8ea41fe4f537c2765e38dcf14
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by zabrocki):

 Replying to [comment:23 darij]:
 > 1) Why are we deprecating `RestrictedPartitions`? From the way it looks,
 this functionality is not implemented elsewhere in its full strength
 (i.e., fixing both a set and a length).

 You may be right about this.  There is a warning note that I may not have
 heeded carefully and I can restore those functions if the current
 `Partitions` function doesn't have this functionality.  I will check
 carefully and restore it if need be.

 > 2) Why are we undeprecating `TransitiveIdeal`? Not deleting it is fine,
 but removing the deprecation warnings?

 The deprecation warnings were only an indication that the function is
 being removed from the global name space since there were previously no
 deprecation warnings if you just imported the functions and started using
 them.  I restored a comment that `TransitiveIdeal` and
 `TransitiveIdealGraded` do need to be deprecated.

 > 3) I have just realized that `CyclicPermutations` of a multiset
 misbehave: e.g., the cardinality method is inherited from `Permutations`
 and thus disagrees with the list method. What should we do about this?
 >
 > 4) Another lost child:
 > {{{
 > sage: Permutations(["c","a","t"])[0]
 > ['c', 'a', 't']
 > sage: _ in Permutations()
 > False
 > }}}
 > (Yet, the class of `Permutations(["c","a","t"])[0]` inherits from
 `Permutations`.)

 Yes.  I agree that both of these are problems.  I tested:
 {{{
 sage:
 len(CyclicPermutations(range(4)))==CyclicPermutations(range(4)).cardinality()
 False
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/19513#comment:24>
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/d/optout.

Reply via email to