#7412 closed defect (duplicate)
from_lehmer_code modifies its argument
Reported by: | ylchapuy | Owned by: | mhansen |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | combinatorics | Keywords: | |
Cc: | hivert | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Here is the problem:
sage: L = [0,0,0] sage: sage.combinat.permutation.from_lehmer_code( L ) [1, 2, 3] sage: L [1, 1, 1]
Attachments (1)
Change History (7)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
- Status changed from new to needs_review
Hi Florent,
Here is the patch. Sorry for the CC, next time I'll follow your advice.
Regards,
Yann
comment:3 Changed 11 years ago by
- Status changed from needs_review to positive_review
comment:4 Changed 11 years ago by
Oups, yes I meant #7411. Thanks for the review.
comment:5 Changed 11 years ago by
- Resolution set to duplicate
- Status changed from positive_review to closed
I don't know if duplicate is the right word, but the patch in #7414 solves this problem as well, but improve also the performance. Sometimes it's useful to think a little bit more...
Yann
comment:6 Changed 11 years ago by
- Milestone changed from sage-combinat to sage-duplicate/invalid/wontfix
Note: See
TracTickets for help on using
tickets.
Hi Yann !
Thanks for the report ! Are you working on it ? The fix is rather trivial and I'm not in the mood for racing with you to get the first patch :-) By the way, thanks for putting me in CC, but it is better to put all the sage-combinat group for all these problem.
Cheers,
Florent