Opened 8 years ago

Closed 8 years ago

#13997 closed defect (fixed)

Fixing a bug in the cyclic sieving phenomenon

Reported by: stumpc5 Owned by: sage-combinat
Priority: major Milestone: sage-5.7
Component: combinatorics Keywords: cyclic sieving phenomenon
Cc: chapoton Merged in: sage-5.7.beta1
Authors: Christian Stump Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by stumpc5)

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]

Attachments (1)

trac_13997-v2.patch (2.4 KB) - added by chapoton 8 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by stumpc5

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

Changed 8 years ago by chapoton

comment:2 Changed 8 years ago by chapoton

  • Status changed from needs_review to positive_review

ok for me. Positive review

comment:3 Changed 8 years ago by chapoton

  • Reviewers set to Frédéric Chapoton

comment:4 Changed 8 years ago by jdemeyer

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