Opened 5 years ago

Closed 5 years ago

#21997 closed defect (fixed)

Fix the check for Littlewood-Richardson tableaux

Reported by: aschilling Owned by:
Priority: major Milestone: sage-7.5
Component: combinatorics Keywords: tableaux, Littlewood-Richardson
Cc: tscrim, j.levinson Merged in:
Authors: Anne Schilling Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: a7a47a1 (Commits, GitHub, GitLab) Commit: a7a47a17dfe3656f1f32e2b61b93d81923af2d8b
Dependencies: Stopgaps:

Status badges

Description

Currently

sage: sage.combinat.lr_tableau.is_littlewood_richardson([[2],[3]],[3,3])
True

whereas this should return False. This is due to the fact that in the code the alphabet of the constructed subwords is not specified.

Change History (5)

comment:1 Changed 5 years ago by aschilling

  • Branch set to public/combinat/LR-fix-21997
  • Cc tscrim j.levinson added; scrim removed
  • Commit set to dfef92ff18c69a05d631cb69916306a3c418d15d
  • Status changed from new to needs_review

New commits:

dfef92ffix to is_littlewood_richardson method

comment:2 Changed 5 years ago by tscrim

  • Reviewers set to Travis Scrimshaw

One little thing for Python3, you should wrap list(range(start, stop)). Once that is changed, you can set a positive review on my behalf.

comment:3 Changed 5 years ago by git

  • Commit changed from dfef92ff18c69a05d631cb69916306a3c418d15d to a7a47a17dfe3656f1f32e2b61b93d81923af2d8b

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

a7a47a1list for range

comment:4 Changed 5 years ago by aschilling

  • Status changed from needs_review to positive_review

comment:5 Changed 5 years ago by vbraun

  • Branch changed from public/combinat/LR-fix-21997 to a7a47a17dfe3656f1f32e2b61b93d81923af2d8b
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.