Opened 10 years ago
Closed 9 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: |
Description (last modified by )
Attachments (1)
Change History (14)
comment:1 Changed 10 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
- Dependencies set to #11440
comment:3 Changed 10 years ago by
comment:4 Changed 10 years ago by
- 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 10 years ago by
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 10 years ago by
- Status changed from needs_review to positive_review
comment:7 Changed 10 years ago by
- Description modified (diff)
- Milestone changed from sage-4.7.1 to sage-4.7.2
comment:8 Changed 10 years ago by
- Milestone changed from sage-4.7.2 to sage-pending
comment:9 Changed 9 years ago by
- Milestone changed from sage-pending to sage-5.4
comment:10 Changed 9 years ago by
- 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 9 years ago by
- 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
Changed 9 years ago by
comment:12 Changed 9 years ago by
- 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 9 years ago by
- Merged in set to sage-5.4.beta2
- Resolution set to fixed
- Status changed from positive_review to closed
don t understand why patchbot still tags this red