#5616: [with patch, needs reivew] speed regression with fast_callable
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: cwitty
Type: defect | Status: assigned
Priority: major | Milestone: sage-3.4.1
Component: basic arithmetic | Keywords:
------------------------------+---------------------------------------------
Changes (by cwitty):
* status: new => assigned
* milestone: sage-3.4.2 => sage-3.4.1
Comment:
The attached patches should fix this problem. (Apply both patches, in
order.)
I split the fix into two logically separate pieces. The first only
generates .pxd files for the fast_callable interpreters; this should have
no observable effect. The second adds a .call_c() cdef method to the
interpreters. (Each interpreter has its own call_c, rather than
inheriting a common method as Robert suggested; I did it this way because
I like typechecking.)
The second patch also modifies parametric_surface.pyx to take advantage of
the call_c() method when generating the surface. I left the old code in
as well; this means you can benchmark fast_eval vs. fast_callable by
setting sage.ext.fast_eval.new_fast_float to False/True.
These patches depend on #5622.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5616#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
