#17435: Cythonise path algebra elements
-------------------------------------+-------------------------------------
Reporter: SimonKing | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.5
Component: algebra | Resolution:
Keywords: path algebra | Merged in:
elements | Reviewers:
Authors: Simon King | Work issues:
Report Upstream: N/A | Commit:
Branch: | 3c8ae84ad88dfa8aab8b2c7a609aaa471a369fae
u/SimonKing/cythonise_path_algebra_elements| Stopgaps:
Dependencies: #16453 |
-------------------------------------+-------------------------------------
Comment (by SimonKing):
I have changed my operating system from openSUSE 12.3 32bit to openSUSE
13.2 64bit. Doing so, I found that one doctest depended on the bits. So, I
fixed that.
Also, I found one crash, and it is astounding that it did not occur
before: When I put a term on the kill list, I dereference the coefficient.
When I remove the kill list (which happens when Sage quits) then I
dereferenced the coefficient *again*. That's of course wrong, and
corrected in the second commit.
Something else: While the change from 32bit to 64bit did not make
letterplace faster, the timings for my implementation improved by as much
as 20%! So, I am looking forward to my applications `:-)`.
Concerning timings: Is it ok to include timings into the docs? After all,
they are on a particular machine etc. Is there a policy? If so, I could
remove the corresponding parts from `sage.quivers.algebra_elements`.
--
Ticket URL: <http://trac.sagemath.org/ticket/17435#comment:18>
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.
