-
Notifications
You must be signed in to change notification settings - Fork 60
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.
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
add isotropic binary morphology functions #421
add isotropic binary morphology functions #421
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Thanks @grlee77 !
rerun tests |
These are based on the distance transform and are a faster way of computing binary morphological operations for large diameter disk or ball footprints. Unlike the sequence footprint decomposition methods, the footprint is exactly circular (spherical).
accidentally had ALL values true rather than ~50% of them
remove unused import
de983ab
to
19080b3
Compare
also test isotropic_opening and isotropic_closing with out kwarg
Not sure why, but use of the out kwarg causes a failure in only one of the CI test runs. I am trying with an explicit copy instead to see if that resolves it. The test case with the issue appears to be:
and then every subsequent test also fails with this same error. |
@gpucibot merge |
related to #419
These are based on the distance transform and are a faster way of computing binary morphological operations for large diameter disk or ball footprints. Unlike the sequence footprint decomposition methods, the footprint is exactly circular (spherical).
One test case
test_isotropic_erosion_spacing
will require #407 to be merged first.#406 will also improve the performance of this implementation