Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Computes the Plancherel measure of an individual partition #11442

Closed
pdehaye mannequin opened this issue Jun 7, 2011 · 15 comments
Closed

Computes the Plancherel measure of an individual partition #11442

pdehaye mannequin opened this issue Jun 7, 2011 · 15 comments

Comments

@pdehaye
Copy link
Mannequin

pdehaye mannequin commented Jun 7, 2011

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

Depends on #11440

Component: combinatorics

Keywords: partition

Author: Paul-Olivier Dehaye

Reviewer: Frédéric Chapoton

Merged: sage-5.4.beta2

Issue created by migration from https://trac.sagemath.org/ticket/11442

@pdehaye pdehaye mannequin added this to the sage-4.7.1 milestone Jun 7, 2011
@pdehaye pdehaye mannequin added c: combinatorics labels Jun 7, 2011
@pdehaye

This comment has been minimized.

@pdehaye pdehaye mannequin added the s: needs review label Jun 7, 2011
@pdehaye
Copy link
Mannequin Author

pdehaye mannequin commented Jun 7, 2011

Dependencies: #11440

@pdehaye
Copy link
Mannequin Author

pdehaye mannequin commented Jun 10, 2011

comment:3

don t understand why patchbot still tags this red

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:4

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

@pdehaye
Copy link
Mannequin Author

pdehaye mannequin commented Jun 10, 2011

Author: Paul-Olivier Dehaye

@pdehaye
Copy link
Mannequin Author

pdehaye mannequin commented Jun 10, 2011

comment:5

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.

@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer modified the milestones: sage-4.7.1, sage-4.7.2 Jun 18, 2011
@jdemeyer jdemeyer removed this from the sage-4.7.2 milestone Jun 27, 2011
@jdemeyer jdemeyer added this to the sage-5.4 milestone Aug 29, 2012
@jdemeyer jdemeyer removed the pending label Aug 29, 2012
@jdemeyer
Copy link

comment:10

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
**********************************************************************

@fchapoton
Copy link
Contributor

comment:11

Here is revised ticket

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

@fchapoton

This comment has been minimized.

@fchapoton
Copy link
Contributor

@fchapoton
Copy link
Contributor

comment:12

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

@fchapoton
Copy link
Contributor

Changed keywords from none to partition

@jdemeyer
Copy link

Merged: sage-5.4.beta2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants