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: |
Description (last modified by )
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)
Change History (5)
comment:1 Changed 8 years ago by
- Description modified (diff)
- Status changed from new to needs_review
Changed 8 years ago by
comment:2 Changed 8 years ago by
- Status changed from needs_review to positive_review
comment:3 Changed 8 years ago by
- Reviewers set to Frédéric Chapoton
comment:4 Changed 8 years ago by
- 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.
ok for me. Positive review