#14862: Compositions accept any input
-------------------------------------+-------------------------------------
Reporter: stumpc5 | Owner: sage-combinat
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.13
Component: combinatorics | Resolution:
Keywords: composition, | Merged in:
FindStat | Reviewers:
Authors: | Work issues: provide a test in the
Report Upstream: N/A | constructor
Branch: | Commit:
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nbruin):
Replying to [comment:10 andrew.mathas]:
> I am against changing the name of the `Partition` class to
`IntegerPartition`. This would create a major patch bomb as `Partition` is
used in many places -- a quick and dirty count with grep gives me 114
different python files.
Such a change wouldn't have to be that invasive. The function
`sage.combinat.partition.Partition` could retain its name and all library
code would be importing that anyway, so no changes are necessary outside
the obviously affected files and the odd doctest. One would just do
{{{
from sage.combinat.partition import Partition as IntegerPartition
}}}
to get the right routine in the global namespace. And perhaps provide a
routine `Partition` to dispatch over different `Partition` types. (I don't
think !Partition is particularly problematic, though).
--
Ticket URL: <http://trac.sagemath.org/ticket/14862#comment:14>
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.
