Re: [numeric] Convolution requires separable filters

From: Mateusz Loskot (mateusz_at_[hidden])
Date: 2019-07-18 22:50:00

On 19-07-15 22:37:27, Mateusz Loskot wrote:
>
>In this thread, I'm archiving some comments related to discussion
>we recently held with Miral on Gitter due to issues with implementation
>of convolution in the current Numeric extension.
>That is, convolve_cols and convolve_rows.
>
>First, reminder that the Numeric implements 2D separable convolution [1].
>More precisely, it is 2D spatial separable convolution [2] and, AFAIU,
>no depth-wise convolving happens.

Somewhat related, I've gone pedantic about correlation and convolution naming

https://github.com/boostorg/gil/pull/339

Please, review in case there is anything to dispute about.

Best regards,
-- 
Mateusz Loskot, http://mateusz.loskot.net
Fingerprint=C081 EA1B 4AFB 7C19 38BA  9C88 928D 7C2A BB2A C1F2