Opened 12 years ago
Last modified 12 years ago
#9250 closed defect
Bug in crystal code — at Version 2
Reported by: | aschilling | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-4.5.2 |
Component: | combinatorics | Keywords: | crystals |
Cc: | combinat | Merged in: | |
Authors: | Anne Schilling | Reviewers: | Daniel Bump |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
This currently breaks:
sage: B=KirillovReshetikhinCrystal(['D',5,1], 3,1) sage: B[0].e(0)
This has to do with the method intermediate_shape for plus/minus diagrams.
Change History (3)
comment:1 Changed 12 years ago by
- Keywords crystals added
- Status changed from new to needs_review
Changed 12 years ago by
comment:2 Changed 12 years ago by
- Description modified (diff)
- Reviewers set to Daniel Bump
- Status changed from needs_review to positive_review
Note: See
TracTickets for help on using
tickets.
After the patch, passes
sage -testall
.Fixes the crash mentioned in the Description.
Includes doctest.
Apparently the intermediate_shape method of PMDiagram presumed that self.n was 2.
This one-line fix is obviously correct.