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.

Sign up for GitHub

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

Jump to bottom

not all TSSCPP are produced #36119

Closed
2 tasks done
mantepse opened this issue Aug 22, 2023 · 0 comments · Fixed by #36124
Closed
2 tasks done

not all TSSCPP are produced #36119

mantepse opened this issue Aug 22, 2023 · 0 comments · Fixed by #36124
Labels
c: combinatorics t: bug
Milestone
sage-10.2

Comments

@mantepse
Copy link
Collaborator

Steps To Reproduce

sage: B = PlanePartitions([6,6,6], symmetry="TSSCPP")
sage: list(B)
[Plane partition [[6, 6, 6, 3, 3, 3], [6, 6, 6, 3, 3, 3], [6, 6, 6, 3, 3, 3], [3, 3, 3], [3, 3, 3], [3, 3, 3]],
 Plane partition [[6, 6, 6, 5, 5, 3], [6, 5, 5, 3, 3, 1], [6, 5, 5, 3, 3, 1], [5, 3, 3, 1, 1], [5, 3, 3, 1, 1], [3, 1, 1]],
 Plane partition [[6, 6, 6, 3, 3, 3], [6, 6, 6, 3, 3, 3], [6, 6, 6, 3, 3, 3], [3, 3, 3], [3, 3, 3], [3, 3, 3]],
 Plane partition [[6, 6, 6, 4, 3, 3], [6, 6, 6, 4, 3, 3], [6, 6, 4, 3, 2, 2], [4, 4, 3, 2], [3, 3, 2], [3, 3, 2]],
 Plane partition [[6, 6, 6, 5, 5, 3], [6, 5, 5, 4, 3, 1], [6, 5, 4, 3, 2, 1], [5, 4, 3, 2, 1], [5, 3, 2, 1, 1], [3, 1, 1]],
 Plane partition [[6, 6, 6, 4, 3, 3], [6, 6, 6, 4, 3, 3], [6, 6, 4, 3, 2, 2], [4, 4, 3, 2], [3, 3, 2], [3, 3, 2]],
 Plane partition [[6, 6, 6, 4, 3, 3], [6, 6, 6, 3, 3, 3], [6, 6, 5, 3, 3, 2], [4, 3, 3, 1], [3, 3, 3], [3, 3, 2]]]
sage: B[0] == B[2]
True

Expected Behavior

There should be seven distinct plane partitions in a box with side length 6.

Actual Behavior

Some plane partitions are missing, others are duplicate.

Additional Information

No response

Environment

irrelevant.

Checklist

  • I have searched the existing issues for a bug report that matches the one I want to file, without success.
  • I have read the documentation and troubleshoot guide
mantepse added a commit to mantepse/sage that referenced this issue Aug 22, 2023
@mantepse
fix cardinality, sagemath#36119, sagemath#36118, sagemath#36116
81b59fa
@mantepse mantepse mentioned this issue Aug 22, 2023
Merged
4 tasks
vbraun pushed a commit to vbraun/sage that referenced this issue Aug 27, 2023
sagemathgh-36124: fix cardinality, sagemath#36119, sagemath#36118, sa…
e38fc5d 
…gemath#36116

    
* Fix a thinko in the iterator for TSSCPP
* make plane partitions hashable
* avoid rational numbers in the cardinality methods, when necessary
* raise an error when an integer and a symmetry is provided as argument
to `PlanePartitions`

Fixes sagemath#36119
Fixes sagemath#36118
Fixes sagemath#36116

- [X] The title is concise, informative, and self-explanatory.
- [X] The description explains in detail what this PR is about.
- [X] I have linked a relevant issue or discussion.
- [X] I have created tests covering the changes.
    
URL: sagemath#36124
Reported by: Martin Rubey
Reviewer(s): Frédéric Chapoton
@mkoeppe mkoeppe added this to the sage-10.2 milestone Aug 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: combinatorics t: bug
Projects
None yet
Milestone
sage-10.2
Development

Successfully merging a pull request may close this issue.

fix cardinality, #36119, #36118, #36116 mantepse/sage
2 participants
@mkoeppe @mantepse