Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#13403 closed enhancement (fixed)

Implementation of bijection between KirillovReshetikhinCrystals and KirillovReshetikhinTableaux

Reported by: tscrim Owned by: tscrim
Priority: major Milestone: sage-5.6
Component: combinatorics Keywords: crystals, rigged configurations
Cc: sage-combinat, aschilling Merged in: sage-5.6.beta2
Authors: Travis Scrimshaw Reviewers: Anne Schilling
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #11305 Stopgaps:

Status badges

Description

Implement the bijection between the usual Kirillov-Reshetikhin crystals and the Kirillov-Reshetikhin tableaux that arise from the bijection with rigged configurations.

Attachments (1)

trac_13403-bijection_KR_crystals_tableaux-ts.patch​ (101.1 KB) - added by tscrim 8 years ago.
Moved broken test to #13460

Download all attachments as: .zip

Change History (14)

comment:1 Changed 9 years ago by tscrim

  • Dependencies changed from 11305 to #11305

comment:2 follow-up: Changed 8 years ago by tscrim

  • Status changed from new to needs_review

comment:3 in reply to: ↑ 2 Changed 8 years ago by aschilling

This patch implements the bijection for type D_n{(1)} between KR crystals and KR tableaux (which form a filling of the Kashiwara-Nakashima tableaux to rectangles). We went through several review cycles by e-mail and the patch looks good to me now.

Positive review! Thanks Travis for your work on this!

Anne

comment:4 Changed 8 years ago by aschilling

  • Keywords rigged configurations added
  • Reviewers set to Anne Schilling
  • Status changed from needs_review to positive_review

comment:5 follow-up: Changed 8 years ago by tscrim

Thank you for the review.

Best,
Travis

comment:6 in reply to: ↑ 5 Changed 8 years ago by aschilling

Replying to tscrim:

Thank you for the review.

Looks good! We discussed this by e-mail!

Anne

comment:7 Changed 8 years ago by jdemeyer

  • Status changed from positive_review to needs_work 
sage -t  -force_lib devel/sage/sage/combinat/rigged_configurations/rigged_configurations.py
**********************************************************************
File "/release/merger/sage-5.6.beta1/devel/sage-main/sage/combinat/rigged_configurations/rigged_configurations.py", line 779:
    sage: ret.to_tensor_product_of_Kirillov_Reshetikhin_crystals()
Expected:
    [[++++, []], [+++-, [[1], [2], [4], [-4]]]]
Got:
    [[++++, []], [+++-, [[1], [2], [3], [-4]]]]
**********************************************************************

Changed 8 years ago by tscrim

Moved broken test to #13460

comment:8 follow-up: Changed 8 years ago by tscrim

  • Status changed from needs_work to needs_review

That test depended on #13460 and has been moved to that patch.

Edit - typo

Last edited 8 years ago by tscrim (previous) (diff)

comment:9 in reply to: ↑ 8 ; follow-up: Changed 8 years ago by aschilling

Replying to tscrim:

That test depended on #13406 and has been moved to that patch.

Do you mean #13460?

comment:10 in reply to: ↑ 9 Changed 8 years ago by tscrim

Replying to aschilling:

Replying to tscrim:

That test depended on #13406 and has been moved to that patch.

Do you mean #13460?

Yes I did.

comment:11 Changed 8 years ago by aschilling

  • Status changed from needs_review to positive_review

comment:12 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.6.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:13 Changed 8 years ago by aschilling

  • Cc sage-combinat added
Note: See TracTickets for help on using tickets.