Skip to content

ENH: Add Python example for ConvolveImageWithKernel#453

Open
hjmjohnson wants to merge 2 commits into
mainfrom
rebase/convolve-image-with-kernel-python
Open

ENH: Add Python example for ConvolveImageWithKernel#453
hjmjohnson wants to merge 2 commits into
mainfrom
rebase/convolve-image-with-kernel-python

Conversation

@hjmjohnson

@hjmjohnson hjmjohnson commented Apr 17, 2026

Copy link
Copy Markdown
Member

Rebase of @PranjalSahu's PR #376 onto current main.

Adds a Python (Code.py) example for ConvolveImageWithKernel, complementing the existing C++ example.

@hjmjohnson hjmjohnson mentioned this pull request Apr 17, 2026
Closed
@github-actions github-actions Bot added type:Enhancement Improvement of existing methods or implementation area:Documentation Issues affecting the Documentation module area:Filtering Issues affecting the Filtering module type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots language:Python Changes to Python examples labels Apr 17, 2026
@hjmjohnson hjmjohnson force-pushed the rebase/convolve-image-with-kernel-python branch from f4ef9ec to 43dee2e Compare June 12, 2026 12:47
dzenanz
dzenanz reviewed Jun 12, 2026
View reviewed changes

@dzenanz dzenanz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good.

Comment thread src/Filtering/Convolution/ConvolveImageWithKernel/MatplotlibVisualizeConvolution.png.sha512
@@ -0,0 +1 @@
082d1d49439879f93946e39675c62a6732fbb1d17aeb7ecbc63c6ca7e350b34416d3764c355e4c1c00583fd8231eb79f01cc6c618d5cda7eee8a267df64db7a3

@dzenanz dzenanz Jun 12, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CID?

@hjmjohnson hjmjohnson force-pushed the rebase/convolve-image-with-kernel-python branch from 43dee2e to 8a3d526 Compare June 13, 2026 03:23
@hjmjohnson hjmjohnson force-pushed the rebase/convolve-image-with-kernel-python branch from 8a3d526 to dafd972 Compare June 13, 2026 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dzenanz dzenanz dzenanz left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area:Documentation Issues affecting the Documentation module area:Filtering Issues affecting the Filtering module language:Python Changes to Python examples type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hjmjohnson @dzenanz @PranjalSahu