#11305 closed enhancement (fixed)
Bijection between Rigged Configurations and Crystal Paths
Reported by: | tscrim | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-5.4 |
Component: | combinatorics | Keywords: | Crystals, days30, days38 |
Cc: | sage-combinat, aschilling | Merged in: | sage-5.4.beta0 |
Authors: | Travis Scrimshaw | Reviewers: | Anne Schilling |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Implementation of rigged configurations and Kirillov-Reshetikhin tableaux and the bijection between the two. Also increases speed of Cartan matrix by saving the result.
Attachments (2)
Change History (13)
comment:1 Changed 7 years ago by
- Keywords days30 added; Days 30 removed
comment:2 Changed 6 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:3 Changed 6 years ago by
comment:4 Changed 6 years ago by
- Keywords days38 added
- Reviewers set to Anne Schilling
- Status changed from needs_review to needs_work
comment:5 follow-up: ↓ 6 Changed 6 years ago by
- Status changed from needs_work to needs_review
Changed 6 years ago by
comment:6 in reply to: ↑ 5 Changed 6 years ago by
Travis incorporated all comments I made above and via e-mail. We tested his implementation of the bijection between rigged configurations and KR tableaux against a Mathematica program written by Reiho Sakamoto.
Once all tests pass, I will set a positive review!
Thanks, Travis, for your work on this!
Anne
comment:7 Changed 6 years ago by
- Description modified (diff)
- Status changed from needs_review to positive_review
comment:8 Changed 6 years ago by
- Milestone changed from sage-5.3 to sage-5.4
comment:9 Changed 6 years ago by
- Merged in set to sage-5.4.beta0
- Resolution set to fixed
- Status changed from positive_review to closed
comment:10 follow-up: ↓ 11 Changed 6 years ago by
My additional patch 11305_long_time.patch needs review (just mention the review in the comments, don't change the status).
comment:11 in reply to: ↑ 10 Changed 6 years ago by
Replying to jdemeyer:
My additional patch 11305_long_time.patch needs review (just mention the review in the comments, don't change the status).
Looks ok to me!
Anne
Hi Travis,
The patch looks close to ready. However, there are still a couple of things that need to be fixed:
Would this be easy to fix? If so, please include it in the patch.
What you call CrystalPaths? is not a generalization of KR crystals, but rather a *different model* or *representation* for KR crystals. This representation at the end should be isomorphic to the model for KR crystals in terms of Kashiwara-Nakashima tableaux. Also your link `sage.combinat.crystals.kirillov_reshetikin' is broken since there is an h missing. I suggest to change this description to
A similar change should be made to
is very implementation oriented. I think a brief mathematical summary would be appropriate. Say what Kleber trees are used for, how the nodes are labelled etc..
So much for now!
Anne