#15439: sage/combinat/matrices/latin.py: isotopism method uses product of
permutations
-------------------------------------------------+-------------------------
Reporter: darij | Owner:
Type: defect | Status:
Priority: major | needs_review
Component: combinatorics | Milestone: sage-5.13
Keywords: permutation, combinat, latin | Resolution:
square | Merged in:
Authors: Nathann Cohen | Reviewers:
Report Upstream: N/A | Work issues:
Branch: u/ncohen/15439 | Commit:
Dependencies: | Stopgaps:
-------------------------------------------------+-------------------------
Changes (by {'newvalue': u'Nathann Cohen', 'oldvalue': ''}):
* status: new => needs_review
* branch: => u/ncohen/15439
* author: => Nathann Cohen
Comment:
Hello ! I tried twice to update the constructor of `Permutation` to add a
`shift_input_by_one=False` flag, but each time it convinced me that it was
bad work. Plus I don't think that this is the good way out -- the good way
out is to have permutations on arbitrary objects.
Hence I did not touch permutation.py. Err, I just changed a few calls to
`all()` to prevent them from building a long useless list of `[False,
False, False, False]`. And I wrote the doc of this isotopism thing, and
avoided this `*` on multiplications.
But honestly, I think you have no way to ensure that nobody will ever use
this `*` in Sage code without worrying over this multiplication order.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/15439#comment:4>
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.
