#15327: More minor tableau and skew_tableau optimizations, and moving out
attacking_pairs
-------------------------------------------------+-------------------------
Reporter: darij | Owner:
Type: enhancement | Status:
Priority: major | needs_review
Component: combinatorics | Milestone: sage-5.13
Keywords: sage-combinat, tableau, | Resolution:
partition, skew tableau | Merged in:
Authors: Darij Grinberg | Reviewers: Travis
Report Upstream: N/A | Scrimshaw
Branch: | Work issues:
Dependencies: #15269 | Commit:
| Stopgaps:
-------------------------------------------------+-------------------------
Changes (by darij):
* status: needs_work => needs_review
Old description:
> This patch brings some little speedups on tableaux and skew tableaux, in
> particular copying over the #15269 fixes to `skew_tableau.py`. It also
> modifies the semantics of the `to_chain` function so as to allow an
> optional `max_entry` parameter (which makes it return a chain of a given
> length -- useful if one is considering semistandard tableaux with ceiling
> higher than the highest entry). Furthermore it moves the
> `attacking_pairs` method from `tableau.py` to `partition.py`, and
> deprecates it in `tableau.py`. In the only place where this method is
> used, it is factored in for speed reasons.
>
> Apply:
>
> * [attachment:trac_15327-qfold-dg.patch]
New description:
This patch brings some little speedups on tableaux and skew tableaux, in
particular copying over the #15269 fixes to `skew_tableau.py`. It also
modifies the semantics of the `to_chain` function so as to allow an
optional `max_entry` parameter (which makes it return a chain of a given
length -- useful if one is considering semistandard tableaux with ceiling
higher than the highest entry). Furthermore it moves the `attacking_pairs`
method from `tableau.py` to `partition.py`, and deprecates it in
`tableau.py`. In the only place where this method is used, it is factored
in for speed reasons.
Apply:
* [attachment:trac_15327-qfold-dg.patch]
* [attachment:trac_15327-fixes-dg.patch]
--
Comment:
I wish it had been just typos. There were two major oopses in my version
of the `inversions` method. Should be fixed now (speed improvement is
still there). Now it probably needs review again, but at least that should
be easy...
for the '''patchbot''':
apply trac_15327-qfold-dg.patch trac_15327-fixes-dg.patch
--
Ticket URL: <http://trac.sagemath.org/ticket/15327#comment:12>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
