#20555: descents for Permutations : cleanup
-------------------------+-------------------------------------------------
Reporter: | Owner:
chapoton | Status: needs_review
Type: | Milestone: sage-7.2
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers:
combinatorics | Work issues:
Keywords: | Commit:
permutation | 6abfeaa8061791dcd8e2ce7066388d39e57ca0e2
Authors: | Stopgaps:
Frédéric Chapoton |
Report Upstream: N/A |
Branch: |
u/chapoton/20555 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by chapoton):
ok, guys. I have added a from_zero argument in the descents of
permutations.
Given that our permutations are indexed by 1,...,n, I really think that
there is no
good reason to have the descents starting at 0. But of course, one need to
be user-friendly, so let us for the moment just prepare for this change.
The branch would work if I disactivate the test for descents in the
testsuites for permutations.
I propose to do that, and also to add a deprecation when using
`from_zero=True`
even if it looks rather strange to me to do that for a default value of
the argument.
Any comments ?
--
Ticket URL: <http://trac.sagemath.org/ticket/20555#comment:12>
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.
