Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#9250 closed defect (fixed)

Bug in crystal code

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

Status badges

Description (last modified by bump)

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 (1)

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

Download all attachments as: .zip

Change History (5)

comment:1 Changed 12 years ago by aschilling

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

Changed 12 years ago by aschilling

comment:2 Changed 12 years ago by bump

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

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 12 years ago by mpatel

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

comment:4 Changed 12 years ago by mvngu

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