Opened 5 years ago

Closed 5 years ago

#19384 closed defect (fixed)

Type B rigged configuration bijection doesn't work when spinor is not the only leftmost factor

Reported by: tscrim Owned by: sage-combinat
Priority: critical Milestone: sage-7.0
Component: combinatorics Keywords: rigged configurations
Cc: sage-combinat, aschilling, bsalisbury1 Merged in:
Authors: Travis Scrimshaw Reviewers: Ben Salisbury
Report Upstream: N/A Work issues:
Branch: b768ea8 (Commits) Commit: b768ea801307b23bb8ca9213c862905a177af982
Dependencies: Stopgaps:

Description

The following breaks:

sage: RC = RiggedConfigurations(['B',3,1], [[3,1],[3,1]])
sage: RC._test_bijection()
sage: RC = RiggedConfigurations(['B',3,1], [[1,1],[3,1],[1,1]])
sage: RC._test_bijection()

This is because we don't keep the vacancy numbers when converting back from the A2n-1(2) rigged configurations and instead recompute them based upon the original dimensions (because we abuse the elements and work with a mutating copy of them to avoid creating numerous temporary parents).

Change History (7)

comment:1 Changed 5 years ago by tscrim

  • Branch set to public/rigged_configurations/fix_type_B_bijection-19384
  • Commit set to 2f7c7277fbc39d44c6a0d4dd56f789c3ed9c78d4
  • Status changed from new to needs_review

comment:2 Changed 5 years ago by git

  • Commit changed from 2f7c7277fbc39d44c6a0d4dd56f789c3ed9c78d4 to 2e6dd67cc26e151af9be36c76bda783294a84c16

Branch pushed to git repo; I updated commit sha1. New commits:

2e6dd67Fixing type B bijection.

comment:3 Changed 5 years ago by tscrim

  • Milestone changed from sage-6.9 to sage-6.10

comment:4 Changed 5 years ago by tscrim

  • Milestone changed from sage-6.10 to sage-7.0

Ping? Should be a quick review.

comment:5 Changed 5 years ago by git

  • Commit changed from 2e6dd67cc26e151af9be36c76bda783294a84c16 to b768ea801307b23bb8ca9213c862905a177af982

Branch pushed to git repo; I updated commit sha1. New commits:

b768ea8Merge branch 'develop' into t/19384/public/rigged_configurations/fix_type_B_bijection-19384

comment:6 Changed 5 years ago by bsalisbury1

  • Reviewers set to Ben Salisbury
  • Status changed from needs_review to positive_review

Thanks, Travis!

comment:7 Changed 5 years ago by vbraun

  • Branch changed from public/rigged_configurations/fix_type_B_bijection-19384 to b768ea801307b23bb8ca9213c862905a177af982
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.