30 Dec
2011
30 Dec
'11
5:13 p.m.
On Fri, Dec 30, 2011 at 8:20 AM, Nadav Horesh <nadavh.horesh@gmail.com> wrote:
Remarks for the convolution subpackage: 1. The __init__.py should be probably changed to
from ext import pyconvolve as convolve
2. The test_convolution.py file is very instructive. It should be polished a bit. I think it would be worthwhile to add also scipy.signal.sepfir2d and cv2.sepFilter2D to the test.
3. I found the convolve speed in par with cv.convolve2d (it can be slower or faster depending on the kernel)
That's great. Do you know how we can reliably make this code compile on all supported platforms? Stéfan
4673
Age (days ago)
4673
Last active (days ago)
0 comments
1 participants
participants (1)
-
Stéfan van der Walt