Skip to content

growth diagrams on compositions#41750

Closed
mantepse wants to merge 22 commits into
sagemath:developfrom
mantepse:u/mantepse/growth_diagrams_on_compositions
Closed

growth diagrams on compositions#41750
mantepse wants to merge 22 commits into
sagemath:developfrom
mantepse:u/mantepse/growth_diagrams_on_compositions

Conversation

@mantepse
Copy link
Copy Markdown
Contributor

@mantepse mantepse commented Mar 4, 2026

This resurrects growth diagrams on compositions

fixes #23941

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 4, 2026

Documentation preview for this PR (built with commit 56556d8; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment on lines +4447 to +4448
The saturated chains in the :meth:`Q_graph` are not familiar to
the author::
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's slightly ambiguous who the "author" is here. Is it you or of those the author(s) of the paper(s) you mentioned above? I would say something more definitive like "This sequence is not currently known by OEIS."

(Side note: It is almost A058122, but just 68 instead of 69. Unfortunately there are not more data points in that entry.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you got me so curious that I installed semigroups and smallsemi, asked an LLM a program to compute it, and made its code work.

Its 68.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The entry in sequence A058122? The entry A058118 has two more numbers: 1376, 7510. They probably are just different generally, but the similarity up until that point was curious.

Comment thread src/sage/combinat/growth.py
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Co-authored-by: Martin Rubey <axiomize@yahoo.de>
Co-authored-by: Travis Scrimshaw <clfrngrown@aol.com>
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py
Comment thread src/sage/combinat/growth.py Outdated
Comment on lines +4447 to +4448
The saturated chains in the :meth:`Q_graph` are not familiar to
the author::
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The entry in sequence A058122? The entry A058118 has two more numbers: 1376, 7510. They probably are just different generally, but the similarity up until that point was curious.

Copy link
Copy Markdown
Collaborator

@tscrim tscrim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last little thing, on top of my previous comments above.

Comment thread src/sage/combinat/growth.py
Co-authored-by: Travis Scrimshaw <clfrngrown@aol.com>
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
mantepse and others added 4 commits April 21, 2026 15:26
Co-authored-by: Travis Scrimshaw <clfrngrown@aol.com>
Co-authored-by: Martin Rubey <axiomize@yahoo.de>
Copy link
Copy Markdown
Collaborator

@tscrim tscrim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. One last blankline you missed and one thing I missed. Once changed, then you can change this to a positive review.

Comment thread src/sage/combinat/growth.py
Comment thread src/sage/combinat/growth.py Outdated
Comment on lines +4453 to +4454
The saturated chains in the :meth:`Q_graph` are not in the OEIS
as of 2026::
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, it is quite easy to explain the numbers. As it turns out, the number of saturated chains to a fixed composition is just the number of standard Young tableaux of the corresponding partition shape. So the total number equals $\sum_{\mu\vdash n} |SYT(\mu)| c_\mu$, where $c_\mu$ is https://www.findstat.org/StatisticsDatabase/St000278/

Should I add this? Note that I did not check it, it's only experimental, although I am completely sure that it is true. I would also guess that it is in the literature, although a popular LLM tells me that this statement is new. Should I add it?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would perhaps add it to FindStat and/or OEIS. You could add it here too, but I don't have a strong opinion either way for that.

Comment thread src/sage/combinat/growth.py Outdated
Comment thread src/sage/combinat/growth.py Outdated
Co-authored-by: Martin Rubey <axiomize@yahoo.de>
@mantepse
Copy link
Copy Markdown
Contributor Author

One final-final thing: in #23941 (comment) you considered various options where to put mason_insert and mason. I am guessing that the best place is rsk.py. I discovered only now that the design there is very similar to the design for the growth diagrams. (N.B., I am still amazed by your skill when you did this redesign, and grateful for it, it works so well!) However, I am a bit afraid that I am opening another week of coding, which I really shouldn't do.

@mantepse
Copy link
Copy Markdown
Contributor Author

I just realised that my reference is wrong. The dual graded graphs were split into a different paper, with only Steph as author. I have to change this.

@mantepse mantepse mentioned this pull request Apr 27, 2026
@mantepse
Copy link
Copy Markdown
Contributor Author

@tscrim I fixed the reference, and renamed mason and mason_insert to underscore methods, to make it clear that they are not intended for public use yet. It seems not completely trivial to merge them into rsk.py, so I'd like to postpone that.

@tscrim
Copy link
Copy Markdown
Collaborator

tscrim commented Apr 27, 2026

In your last push, they are not underscored. Is this coming on an forthcoming push (or gobbled in a merge)?

@mantepse
Copy link
Copy Markdown
Contributor Author

Thank you for looking! Some things I'll never understand :-)

@mantepse
Copy link
Copy Markdown
Contributor Author

Dear @tscrim, sorry for any confusion: can I set this to positive already?

@mantepse mantepse mentioned this pull request May 9, 2026
@mantepse
Copy link
Copy Markdown
Contributor Author

superseded by #42079

@mantepse mantepse closed this May 18, 2026
vbraun pushed a commit that referenced this pull request May 29, 2026
gh-42079: Growth/testsuite
    
To facilitate checking backward and forward rules, we check the forced
cases.

Supersedes #42093 and #41750
    
URL: #42079
Reported by: Martin Rubey
Reviewer(s): Frédéric Chapoton, Martin Rubey
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

growth diagrams on compositions

2 participants