Refactor gaussian_blur in numpy to utilize scipy.signal.convolve2d #20974
+15
−25
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To improve performance, I optimized the convolution logic in gaussian_blur by replacing the NumPy-based implementation with scipy.signal.convolve2d.
Below are the performance test results on my local machine:
CPU times: user 7.59 s, sys: 0 ns, total: 7.59 s
Wall time: 7.59 s
CPU times: user 123 ms, sys: 0 ns, total: 123 ms
Wall time: 122 ms