Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#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 Merged in: sage-4.8.alpha0
Authors: Anne Schilling, Mike Zabrocki Reviewers: Nicolas Thiery, Mike Zabrocki
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #11700, #10530 Stopgaps:

Description (last modified by aschilling)

This patch implements a class for cores and methods associated to them.


Apply trac_11742-cores-as.patch

Attachments (1)

trac_11742-cores-as.patch (30.1 KB) - added by aschilling 6 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 6 years ago by aschilling

  • Dependencies set to #11700
  • Reviewers set to Nicolas Thiery, Mike Zabrocki
  • Status changed from new to needs_review

comment:2 Changed 6 years ago by aschilling

  • Component changed from PLEASE CHANGE to combinatorics

comment:3 Changed 6 years ago by aschilling

  • Type changed from PLEASE CHANGE to enhancement

comment:4 Changed 6 years 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:5 Changed 6 years ago by aschilling

  • Description modified (diff)

comment:6 Changed 6 years ago by aschilling

  • Dependencies changed from #11700 to #11700, #10530

comment:7 Changed 6 years ago by aschilling

  • Description modified (diff)

comment:8 follow-up: Changed 6 years 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

Changed 6 years ago by aschilling

comment:9 in reply to: ↑ 8 Changed 6 years 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:10 Changed 6 years ago by jdemeyer

  • Milestone changed from sage-4.7.2 to sage-4.7.3

comment:11 Changed 6 years ago by jdemeyer

  • Merged in set to sage-4.7.3.alpha0
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:12 Changed 6 years ago by jdemeyer

  • Milestone sage-4.7.3 deleted

Milestone sage-4.7.3 deleted

comment:13 Changed 6 years ago by jdemeyer

  • Merged in changed from sage-4.7.3.alpha0 to sage-4.8.alpha0
  • Milestone set to sage-4.8
Note: See TracTickets for help on using tickets.