#6985: [with patch, needs work] complex_plot needs to use fast_callable
----------------------+-----------------------------------------------------
Reporter: jason | Owner: mhansen
Type: defect | Status: assigned
Priority: major | Milestone: sage-4.1.2
Component: graphics | Keywords:
Reviewer: | Author: Mike Hansen
Merged: |
----------------------+-----------------------------------------------------
Comment(by jason):
There seems to be a regression: {{{%time complex_plot(exp(x)-sin(x), (-10,
10), (-10, 10))}}} takes 21 seconds before the patch, but 28 seconds after
the patch for me.
Note:
{{{
sage: f(x)=exp(x)-sin(x)
sage: fcomplex=fast_callable(f, domain=complex, expect_one_var=True)sage:
fCDF=fast_callable(f, domain=CDF, expect_one_var=True)
sage: %timeit f(4j)
100 loops, best of 3: 2.21 ms per loop
sage: %timeit fcomplex(4j)
100 loops, best of 3: 2.7 ms per loop
sage: %timeit fCDF(4j)
100000 loops, best of 3: 7.94 µs per loop
}}}
So maybe the fast_callable in the patch should use domain CDF!
(this seems really odd to me, but I can't argue with the timings above!)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6985#comment:5>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
