#13605: Partition options and cleanup partitions documentation
------------------------------------------------------+---------------------
Reporter: tscrim | Owner:
sage-combinat
Type: enhancement | Status:
needs_review
Priority: major | Milestone:
sage-5.7
Component: combinatorics | Resolution:
Keywords: partition, options, output, days45 | Work issues:
Report Upstream: N/A | Reviewers: Andew
Mathas
Authors: Travis Scrimshaw | Merged in:
Dependencies: #13074 #13762 #13840 #14065 | Stopgaps:
------------------------------------------------------+---------------------
Changes (by andrew.mathas):
* keywords: partition, options, output => partition, options, output,
days45
Comment:
The patch is good except for two TestSuite failures which comes down to
the following:
{{{
sage: [5] in Partitions(5, min_part=2)
True
sage: Partition([5]) in Partitions(5, min_part=2)
False
sage: [3,1,1] in OrderedPartitions(5,3)
True
sage: Partition([3,1,1]) in OrderedPartitions(5,3)
False
}}}
They look like they are both the same issue with IntegerListsLex, but I
couldn't see how to fix this. Once these problems are fixed -- and once
some one reviews my review patch -- I am happy to give this a positive
review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13605#comment:36>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.