confusing comment in Gaussian 1D kernel creation
easz opened this issue · comments
tsungi commented
detectree/detectree/filters.py
Line 15 in 56e8f30
# # Since we are calling correlate, not convolve, revert the kernel
# weights = _gaussian_kernel1d(sigma, order, lw)[::-1]
weights = _gaussian_kernel1d(sigma, order, lw)
the kernel is not reverted or do I understand wrong?
Martí Bosch commented
Hello @easz,
this was written a while ago and I do not remember exactly why the kernel was reverted at first, but in the released code the kernel is not reverted (the [::-1]
is not in the uncommented line), and it is actually ndi.convolve
that is used to build the pixel features with the kernel.
Hope this helps. Best,
Martí