Ticket #11742 (closed enhancement: fixed)
Implementation of class for cores
| Reported by: | aschilling | Owned by: | tbd |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.8 |
| Component: | combinatorics | Keywords: | cores, partitions |
| Cc: | sage-combinat | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Nicolas Thiery, Mike Zabrocki |
| Authors: | Anne Schilling, Mike Zabrocki | Merged in: | sage-4.8.alpha0 |
| Dependencies: | #11700, #10530 | Stopgaps: |
Description (last modified by aschilling) (diff)
This patch implements a class for cores and methods associated to them.
Attachments
-
trac_11742-cores-as.patch
(30.1 KB) -
added by aschilling 21 months ago.
Change History
comment:1 Changed 22 months ago by aschilling
- Status changed from new to needs_review
- Reviewers set to Nicolas Thiery, Mike Zabrocki
- Dependencies set to #11700
comment:4 Changed 22 months ago by aschilling
The patch is ready for review. Nicolas already gave a lot of advice on the design and wrote a review patch, which has been incorporated in the posted patch.
comment:8 follow-up: ↓ 9 Changed 21 months ago by zabrocki
- Status changed from needs_review to positive_review
Everything that I ask for was addressed. Thanks Anne! Chris and I were using this code just yesterday and found it super useful. I double checked that the transpositions were on the correct side :
sage: c = Core([11,8,5,5,3,3,1,1,1],4) sage: c.affine_symmetric_group_action([2,5],transposition=True) [11, 8, 7, 6, 5, 4, 3, 2, 1]
which agrees with Example 9.6 of LLMS - Affine Insertion paper
comment:9 in reply to: ↑ 8 Changed 21 months ago by aschilling
Replying to zabrocki:
Everything that I ask for was addressed. Thanks Anne! Chris and I were using this code just yesterday and found it super useful. I double checked that the transpositions were on the correct side :
sage: c = Core([11,8,5,5,3,3,1,1,1],4) sage: c.affine_symmetric_group_action([2,5],transposition=True) [11, 8, 7, 6, 5, 4, 3, 2, 1]which agrees with Example 9.6 of LLMS - Affine Insertion paper
Thanks! I added this test to the patch as well!
Anne
comment:11 Changed 21 months ago by jdemeyer
- Status changed from positive_review to closed
- Resolution set to fixed
- Merged in set to sage-4.7.3.alpha0
comment:12 Changed 20 months ago by jdemeyer
- Milestone sage-4.7.3 deleted
Milestone sage-4.7.3 deleted
comment:13 Changed 20 months ago by jdemeyer
- Merged in changed from sage-4.7.3.alpha0 to sage-4.8.alpha0
- Milestone set to sage-4.8
