sage icon indicating copy to clipboard operation
sage copied to clipboard

add add_entry and anti_restrict methods to SkewTableau

Open jmcdonough98 opened this issue 8 months ago • 4 comments

add_entry and anti_restrict are already existing methods in Tableau, but they also make sense for SkewTableau. This PR ports these methods to SkewTableau, and adds a simple check to add_entry to avoid trying to construct a new tableau when the given cell is too large.

:memo: Checklist

  • [x] The title is concise and informative.
  • [x] The description explains in detail what this PR is about.
  • [ ] I have linked a relevant issue or discussion.
  • [x] I have created tests covering the changes.
  • [x] I have updated the documentation and checked the documentation preview.

:hourglass: Dependencies

None

jmcdonough98 avatar Apr 09 '25 19:04 jmcdonough98

@jmcdonough98 as long as your tests pass locally, you can set to positive review

trevorkarn avatar May 14 '25 21:05 trevorkarn

@trevorkarn all tests pass on my machine, but github is not letting me change PR labels. I think I don't have permission.

jmcdonough98 avatar May 14 '25 22:05 jmcdonough98

merge conflict, please fix

vbraun avatar May 18 '25 16:05 vbraun

Thanks @vbraun. @jmcdonough98 Do you need help with this?

trevorkarn avatar May 18 '25 16:05 trevorkarn