Opened 9 years ago

Closed 8 years ago

#11442 closed enhancement (fixed)

Computes the Plancherel measure of an individual partition

Reported by: pdehaye Owned by: sage-combinat
Priority: major Milestone: sage-5.4
Component: combinatorics Keywords: partition
Cc: Merged in: sage-5.4.beta2
Authors: Paul-Olivier Dehaye Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #11440 Stopgaps:

Attachments (1)

trac_11442-plancherel_measure_partition-pod.v2.patch (2.7 KB) - added by chapoton 8 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 9 years ago by pdehaye

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 9 years ago by pdehaye

  • Dependencies set to #11440

comment:3 Changed 9 years ago by pdehaye

don t understand why patchbot still tags this red

comment:4 Changed 9 years ago by chapoton

  • Reviewers set to Frédéric Chapoton

For me, this is positive review, but I would like that the bot says "green" before going on.

Please add your name to the field Author

This is strange indeed that it stays red. Let us try to restart the bot :

Apply trac_11442-plancherel_measure_partition-pod.patch

comment:5 Changed 9 years ago by pdehaye

  • Authors set to Paul-Olivier Dehaye

thanks. according to Robert Bradshaw (privately), it is due to the fact that it was not anticipated the exact same patch would be run more than once on the same machine, so it goes with the worst result (which in this case it shouldn't).

this happened because i had the dependencies of dependencies wrong the first time around, so he manually asked the patchbot to restest the exact same patch.

comment:6 Changed 9 years ago by chapoton

  • Status changed from needs_review to positive_review

comment:7 Changed 9 years ago by jdemeyer

  • Description modified (diff)
  • Milestone changed from sage-4.7.1 to sage-4.7.2

comment:8 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-4.7.2 to sage-pending

comment:9 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-pending to sage-5.4

comment:10 Changed 8 years ago by jdemeyer

  • Status changed from positive_review to needs_work

I get a doctest failure

sage -t  -force_lib devel/sage/sage/combinat/partition.py
**********************************************************************
File "/release/merger/sage-5.4.beta1/devel/sage-main/sage/combinat/partition.py", line 3060:
    sage: RestrictedPartitions(8,[1,3,5,7]).list()
Expected:
    doctest:...: DeprecationWarning: RestrictedPartitions is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    [[7, 1], [5, 3], [5, 1, 1, 1], [3, 3, 1, 1], [3, 1, 1, 1, 1, 1], [1, 1, 1, 1, 1, 1, 1, 1]]
Got:
    doctest:1: DeprecationWarning: RestrictedPartitions is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    doctest:2988: DeprecationWarning: RestrictedPartitions_nsk is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    [[7, 1], [5, 3], [5, 1, 1, 1], [3, 3, 1, 1], [3, 1, 1, 1, 1, 1], [1, 1, 1, 1, 1, 1, 1, 1]]
**********************************************************************
File "/release/merger/sage-5.4.beta1/devel/sage-main/sage/combinat/partition.py", line 3085:
    sage: RestrictedPartitions(8,[1,3,5,7]).cardinality()
Expected:
    doctest:...: DeprecationWarning: RestrictedPartitions is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    doctest:...: DeprecationWarning: RestrictedPartitions_nsk is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    6
Got:
    doctest:1: DeprecationWarning: RestrictedPartitions is deprecated; use Partitions with the parts_in keyword instead.
    See http://trac.sagemath.org/5478 for details.
    6
**********************************************************************

comment:11 Changed 8 years ago by chapoton

  • Description modified (diff)
  • Status changed from needs_work to needs_review

Here is revised ticket

apply only: trac_11442-plancherel_measure_partition-pod.v2.patch

comment:12 Changed 8 years ago by chapoton

  • Keywords partition added
  • Status changed from needs_review to positive_review

Applies smoothly on 5.4beta1, all test pass, doc is ok: positive review

comment:13 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.4.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.