search

image-rs / imageproc

Image processing operations
MIT License
758 stars 150 forks source link

Fix lalplacian #650

Closed cospectrum closed 6 months ago

cospectrum commented 6 months ago

Possible fix of https://github.com/image-rs/imageproc/issues/649 ?

theotherphil commented 6 months ago

Can you add a test as well to prevent any future refactoring errors.

theotherphil commented 6 months ago

I was imagining a test of the outputs but I guess this works too!

Thanks for the fix. I intended to add a test after merging the initial PR for Laplacian filtering but forgot about it in all the follow up discussion. I’ll make sure all functions have tests pre-merge in future.