#11115: Rewrite cached_method in Cython
---------------------------+------------------------------------------------
Reporter: SimonKing | Owner: jason
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7
Component: misc | Keywords: category cython cache
Work_issues: | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies:
---------------------------+------------------------------------------------
Comment(by novoselt):
I think that the current behaviour of the patch regarding cache storage is
just fine. Getting more memory tends to be much easier and cheaper than
getting a faster CPU.
I was avoiding using `@cached_method` before due to its issues with speed
and documentation, but have started using it recently and it is so
convenient! The more general, convenient, and well-behaved these
decorators are, the more developers will adapt them.
While in principle it is not too much of a hassle to insert an extra
attribute into your new class, it is quite annoying to always remember
that if you decide to put a decorator on some method you have also to
check if there is that extra attribute already defined or not. And as I
understand, things don't break if you forget to do it, they just work
slower than they should.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11115#comment:28>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
