Ticket #9250 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Bug in crystal code

Reported by: aschilling Owned by: sage-combinat
Priority: major Milestone: sage-4.5.2
Component: combinatorics Keywords: crystals
Cc: combinat Work issues:
Report Upstream: N/A Reviewers: Daniel Bump
Authors: Anne Schilling Merged in: sage-4.5.2.alpha0
Dependencies: Stopgaps:

Description (last modified by bump) (diff)

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.

Attachments

trac_9250-crystalbug-as.patch Download (1.9 KB) - added by aschilling 3 years ago.

Change History

comment:1 Changed 3 years ago by aschilling

  • Keywords crystals added
  • Status changed from new to needs_review
  • Authors set to Anne Schilling

Changed 3 years ago by aschilling

comment:2 Changed 3 years ago by bump

  • Status changed from needs_review to positive_review
  • Reviewers set to Daniel Bump
  • Description modified (diff)

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.

comment:3 Changed 3 years ago by mpatel

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

comment:4 Changed 3 years ago by mvngu

  • Milestone set to sage-4.5.2
Note: See TracTickets for help on using tickets.