Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSTPosets are lattices #29525

Closed
darijgr opened this issue Apr 17, 2020 · 8 comments
Closed

SSTPosets are lattices #29525

darijgr opened this issue Apr 17, 2020 · 8 comments

Comments

@darijgr
Copy link
Contributor

darijgr commented Apr 17, 2020

https://doc.sagemath.org/html/en/reference/combinat/sage/combinat/posets/poset_examples.html#sage.combinat.posets.poset_examples.Posets.SSTPoset:

static SSTPoset(s, f=None)

The poset on semistandard tableaux of shape s and largest entry f that is ordered by componentwise comparison of the entries.

This isn't just some finite poset; it's a lattice, with minimum and maximum being defined entrywise. That should be in the code and in the doc.

Any volunteers? (My Sage isn't working and I have a long to-do list of papers right now, so I pass.)

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: lattices, posets, tableaux, sage-combinat

Author: Frédéric Chapoton

Branch/Commit: de3ea1f

Reviewer: Darij Grinberg

Issue created by migration from https://trac.sagemath.org/ticket/29525

@darijgr darijgr added this to the sage-9.1 milestone Apr 17, 2020
@fchapoton
Copy link
Contributor

comment:1

EDIT: sorry, wrong comparison algorithm

@fchapoton
Copy link
Contributor

Commit: de3ea1f

@fchapoton
Copy link
Contributor

Author: Frédéric Chapoton

@fchapoton
Copy link
Contributor

Branch: u/chapoton/29525

@fchapoton
Copy link
Contributor

comment:2

and now, who is going to review that ?


New commits:

de3ea1ftrac #29525 sstposets as lattices

@darijgr
Copy link
Contributor Author

darijgr commented Apr 18, 2020

comment:3

LGTM!

@darijgr
Copy link
Contributor Author

darijgr commented Apr 18, 2020

Reviewer: Darij Grinberg

@vbraun
Copy link
Member

vbraun commented Apr 22, 2020

Changed branch from u/chapoton/29525 to de3ea1f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants