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

Fixing a bug in the cyclic sieving phenomenon #13997

Closed
stumpc5 opened this issue Jan 23, 2013 · 5 comments
Closed

Fixing a bug in the cyclic sieving phenomenon #13997

stumpc5 opened this issue Jan 23, 2013 · 5 comments

Comments

@stumpc5
Copy link
Contributor

stumpc5 commented Jan 23, 2013

This patch fixes a bug in the cyclic sieving phenomenon.

Without the patch applied, we have

sage: from sage.combinat.cyclic_sieving_phenomenon import CyclicSievingPolynomial
sage: S42 = [ Set(S) for S in subsets([1,2,3,4]) if len(S) == 2 ]; S42
sage: cyc_act = lambda S: Set( i.mod(4)+1 for i in S)
sage: CyclicSievingPolynomial( S42, cyc_act, order=8, get_order=True )
[q^6 + 2*q^4 + q^2 + 2, 4]

while with the patch applied, you get

sage: CyclicSievingPolynomial( S42, cyc_act, order=8, get_order=True )
[q^6 + 2*q^4 + q^2 + 2, 8]

CC: @fchapoton

Component: combinatorics

Keywords: cyclic sieving phenomenon

Author: Christian Stump

Reviewer: Frédéric Chapoton

Merged: sage-5.7.beta1

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

@stumpc5

This comment has been minimized.

@fchapoton
Copy link
Contributor

Attachment: trac_13997-v2.patch.gz

@fchapoton
Copy link
Contributor

comment:2

ok for me. Positive review

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@jdemeyer
Copy link

Merged: sage-5.7.beta1

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

3 participants