New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Optimized Box Blur for different x-y radii #2795

Merged

joeyballentine merged 1 commit into chaiNNer-org:main from RunDevelopment:box-blur-xy-opt

Apr 15, 2024

Member

RunDevelopment commented Apr 15, 2024

Changes:

A box blur only has no effect is the both radii are 0, not when just one 1. I fixed this, which allows users to blur solely in the X or Y direction now.
Round large radii to integer independently of each other.
Use very fast cv2.blur if possible when separating kernel.

These optimizations make certain edges cases more than 10x faster.

Before:

After:


          Optimized Box Blur for different x-y radii

f567433

joeyballentine approved these changes

View reviewed changes

joeyballentine merged commit 60a0bb5 into chaiNNer-org:main

14 checks passed

RunDevelopment deleted the box-blur-xy-opt branch

April 15, 2024 15:57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment