#19737: SetPartition.to_permutation().cycle_tuples() is not the identity
-------------------------------------+-------------------------------------
Reporter: mantepse | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-7.0
Component: combinatorics | Resolution:
Keywords: set partitions, | Merged in:
permutations | Reviewers: Nathann Cohen
Authors: Martin Rubey | Work issues:
Report Upstream: N/A | Commit:
Branch: | c6a3939d2e18efe3906e1f959dc5c019c921a944
u/mantepse/setpartition_to_permutation___cycle_tuples___is_not_the_identity|
Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by ncohen):
* reviewer: => Nathann Cohen
Comment:
> Well, that's why I hate python.
You hate Python because the combinat folks break name conventions? `O_o`
> A better solution (in my opinion) would be to have conditionally
exported methods.
What do you mean by that? Can you give an example?
> Or, lacking this, the possibility to have a function and a class of the
same name, eg., {{{SetPartition}}} would be a function that return an
instance of a class of the *same* name, or an instance of
{{{IntegerSetPartition}}}.
That's what I call making things complicated. Can't people live with
classes that return instances of themselves?
> Can you give this ticket positive review now?
I added a small commit at public/19737. If you agree with it, please add
it to this ticket and set the status to `positive_review`.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/19737#comment:28>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
