#15143: Improvements to SetPartitions
------------------------------------+-----------------------------
Reporter: tscrim | Owner: sage-combinat
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.12
Component: combinatorics | Resolution:
Keywords: | Merged in:
Authors: Travis Scrimshaw | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: #14234 | Stopgaps:
------------------------------------+-----------------------------
Comment (by darij):
Hi Travis,
thanks for the example; I didn't expect that {{{A_i}}} would really mean
the i-th part of A (with A being unordered, this sounded a bit arbitrary
to me). I think I've made the doc for this method a lot more readably now
(there was some index confusion with both A and s having the same length
q, and a condition r < |S| which I think is completely out of place).
Please also look at the pipe method I've added (mainly because its
docstring clarifies the notation used in other methods). If you agree with
Mike's and my changes, this is a positive_review.
Best regards,
Darij
--
Ticket URL: <http://trac.sagemath.org/ticket/15143#comment:13>
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.
