#16472: get rid of PermutationsNK
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-6.3
Component: combinatorics | Resolution:
Keywords: | Merged in:
Authors: Vincent Delecroix | Reviewers: Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/remove_PermutationsNK_pickling-16472|
c86ef49606433e531fafe40bc301714efb1eb64c
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by vdelecroix):
* status: needs_review => positive_review
Comment:
Hey Travis,
Your version is definitely better. Thanks. But I got
{{{
sage -t --long combinat/permutation.py
**********************************************************************
File "combinat/permutation.py", line 7458, in
sage.combinat.permutation.PermutationsNK.__setstate__
Failed example:
...
SyntaxError: EOL while scanning string literal
}}}
It is because of a backslash that escaped a `"`. I put a commit for this
and now everything is fine so I set to positive review.
Thanks for your help.
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/16472#comment:11>
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/d/optout.
