Need an advice

[Stéfan van der Walt]

On Fri, Dec 30, 2011 at 8:20 AM, Nadav Horesh <nadavh.horesh at 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