Opened 10 years ago
Closed 9 years ago
#14101 closed enhancement (fixed)
Remove CombinatorialClass from skew*
Reported by: | tscrim | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-5.12 |
Component: | combinatorics | Keywords: | days45, days49 |
Cc: | sage-combinat, zabrocki, nthiery, alubovsky | Merged in: | sage-5.12.beta4 |
Authors: | Travis Scrimshaw, Arthur Lubovsky | Reviewers: | Travis Scrimshaw, Arthur Lubovsky |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #7983, #14772, #13589, #14907, #10630 | Stopgaps: |
Description (last modified by )
Removing CombinatorialClass
from combinat/skew_partition.py
and combinat/skew_tableau.py
.
Part of #12913.
Attachments (2)
Change History (31)
comment:1 Changed 10 years ago by
Component: | PLEASE CHANGE → combinatorics |
---|---|
Keywords: | sage-combinat removed |
comment:2 Changed 10 years ago by
Authors: | Travis Scrimshaw → Travis Scrimshaw, Arthur Lubovsky |
---|---|
Cc: | nthiery alubovsky added |
Dependencies: | → #14772 |
Keywords: | days49 added |
Reviewers: | → Travis Scrimshaw, Arthur Lubovsky |
Status: | new → needs_review |
comment:3 Changed 10 years ago by
Dependencies: | #14772 → #14772 #13589 |
---|
There is a minor dependency in integer_vector_weighted.py
from #13589.
comment:4 Changed 10 years ago by
Dependencies: | #14772 #13589 → #14772 #13589 #14907 |
---|
Removed fuzz caused by #14907 (I think).
comment:6 Changed 10 years ago by
Cc: | zabrocki added |
---|
comment:10 Changed 10 years ago by
Status: | needs_review → positive_review |
---|
comment:11 Changed 10 years ago by
1) Why does this depend on #14907?
2)
sage -t devel/sage-main/sage/combinat/ribbon_tableau.py ********************************************************************** File "devel/sage-main/sage/combinat/ribbon_tableau.py", line 201, in sage.combinat.ribbon_tableau.RibbonTableaux Failed example: for i in R: i.pp(); print Expected: . . 0 0 0 . 0 0 2 1 0 1 Got: . . 0 0 0 . 0 0 2 1 0 1 <BLANKLINE> . . 1 0 0 . 0 0 0 1 0 2 <BLANKLINE> . . 0 0 0 . 1 0 1 2 0 0 <BLANKLINE> **********************************************************************
This seems to be a bug with the doctesting framework, which thinks of empty lines as EOM. Just replacing "print" by "print 'some_nonempty_string'" does the trick.
comment:12 Changed 10 years ago by
Status: | positive_review → needs_work |
---|
comment:13 Changed 10 years ago by
Description: | modified (diff) |
---|---|
Status: | needs_work → needs_review |
Fixed the above (all one has to do was add <BLANKLINE>
) and added the comment that Arthur and I talked about via email.
For patchbot:
Apply: trac_14101-remove_cc_skew-ts.patch
comment:15 Changed 10 years ago by
Arthur had wanted me to change some documentation, so he'll set the positive review once he's looked that over.
comment:16 Changed 10 years ago by
Status: | needs_review → positive_review |
---|
comment:19 Changed 10 years ago by
Milestone: | sage-5.11 → sage-5.12 |
---|
comment:20 Changed 10 years ago by
Status: | positive_review → needs_work |
---|
There is fuzz applying to 5.11.beta3
applying trac_14101-remove_cc_skew-ts.patch patching file sage/combinat/tableau.py Hunk #10 succeeded at 1486 with fuzz 2 (offset -52 lines). now at: trac_14101-remove_cc_skew-ts.patch
None of the dependencies touch tableau.py. Can you please rebase to see if that will fix the fuzz?
Thanks, Jeff
comment:21 Changed 10 years ago by
Dependencies: | #14772 #13589 #14907 → #7983 #14772 #13589 #14907 |
---|---|
Status: | needs_work → positive_review |
Try it with #7983. I'm not getting any fuzz (in the combinat queue).
comment:22 Changed 10 years ago by
Travis,
Yes that worked. It now applies without any fuzz. Thanks.
Jeff
comment:24 Changed 10 years ago by
Status: | positive_review → needs_work |
---|
I get a warning when building the doc:
[combinat ] /Applications/sage/local/lib/python2.7/site-packages/sage/combinat/ribbon_tableau.py:docstring of sage.combinat.ribbon_tableau.RibbonTableau:23: WARNING: Literal block expected; none found.
comment:25 Changed 10 years ago by
Status: | needs_work → positive_review |
---|
I've fixed the offending docstring.
For patchbot:
Apply: trac_14101-remove_cc_skew-ts.patch
comment:26 Changed 10 years ago by
Dependencies: | #7983 #14772 #13589 #14907 → #7983, #14772, #13589, #14907, #10630 |
---|---|
Milestone: | sage-5.12 → sage-pending |
comment:27 Changed 10 years ago by
Milestone: | sage-pending → sage-5.12 |
---|
Changed 10 years ago by
Attachment: | trac_14101-remove_cc_skew-ts.patch added |
---|
comment:29 Changed 9 years ago by
Merged in: | → sage-5.12.beta4 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
The dependency on #14772 is some fuzz due to editing of cython files in symmetrica.