#4206: [with patch, positive review] convert RDF and CDF vectors to use numpy
----------------------------+-----------------------------------------------
Reporter: jason | Owner: jason
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-3.2.2
Component: linear algebra | Resolution:
Keywords: |
----------------------------+-----------------------------------------------
Changes (by jason):
* summary: [with patch, needs review] convert RDF and CDF vectors to use
numpy => [with patch, positive review] convert
RDF and CDF vectors to use numpy
Comment:
Okay, updated the vector_fft.patch. The new timing comparison is:
{{{
sage: v=vector(RDF, range(1000000r))
n=v.numpy()
sage: n=v.numpy()
sage: import scipy; import scipy.fftpack
sage: timeit('a=v.fft()')
5 loops, best of 3: 342 ms per loop
sage: timeit('b=scipy.fft(n)')
5 loops, best of 3: 316 ms per loop
sage: timeit('a=v.inv_fft()')
5 loops, best of 3: 442 ms per loop
sage: timeit('b=scipy.ifft(n)')
5 loops, best of 3: 416 ms per loop
sage: v=vector(CDF, range(1000000r))
sage: n=v.numpy()
sage: timeit('a=v.fft()')
5 loops, best of 3: 346 ms per loop
sage: timeit('b=scipy.fft(n)')
5 loops, best of 3: 319 ms per loop
sage: timeit('a=v.inv_fft()')
5 loops, best of 3: 470 ms per loop
sage: timeit('b=scipy.ifft(n)')
5 loops, best of 3: 419 ms per loop
}}}
Since the timing issue is addressed, I'm marking this as positive review
as directed in jkantor's comment.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4206#comment:43>
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
-~----------~----~----~----~------~----~------~--~---