#16453: Cythonize quiver paths
-------------------------------------+-------------------------------------
Reporter: SimonKing | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.4
Component: algebra | Resolution:
Keywords: | Merged in:
Authors: Simon King | Reviewers:
Report Upstream: N/A | Work issues: Fix issue in bounded
Branch: | integer sequences
public/ticket/16453 | Commit:
Dependencies: #15820 | 2abe9e77963283f9b4034316d824ae84bbbdeebc
| Stopgaps:
-------------------------------------+-------------------------------------
Comment (by SimonKing):
Replying to [comment:67 chapoton]:
> One failing doctest here:
> {{{
> File "src/sage/quivers/paths.pyx", line 773, in
sage.quivers.paths.NewQuiverPath
> Failed example:
> p.__reduce__()
> Expected:
> (<...NewQuiverPath>,
> (Partial semigroup formed by the directed paths of Multi-digraph on
3 vertices,
> 1,
> 3,
> (0, 4L, 1, 4, (4L,)),
> 2L,
> 2))
> Got:
> (<built-in function NewQuiverPath>,
> (Partial semigroup formed by the directed paths of Multi-digraph on
3 vertices,
> 1,
> 3,
> (0, 4L, 1, 8, (4L,)),
> 2L,
> 2))
> }}}
Did I not correct that error?
Too bad, it could be that I corrected it in #17435, since I had the
impression I had introduced. Well, with mercurial we could now simply move
the patch from there to here...
The explanation of above error is of course that I made the tests on a
32bit machine, but you have 64bit. Meanwhile, I have switched to 64bit
too, and at that point I corrected the error (but on the wrong ticket, I
am afraid).
--
Ticket URL: <http://trac.sagemath.org/ticket/16453#comment:69>
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.
