Skip to content
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

Conversation

grlee77
Copy link
Contributor

@grlee77 grlee77 commented Nov 1, 2022

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

@grlee77 grlee77 added feature request New feature or request non-breaking Introduces a non-breaking change labels Nov 1, 2022
@grlee77 grlee77 added this to the v22.12.00 milestone Nov 1, 2022
@grlee77 grlee77 requested a review from a team as a code owner November 1, 2022 17:04
Copy link
Contributor

@gigony gigony left a 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 !

@grlee77
Copy link
Contributor Author

grlee77 commented Nov 29, 2022

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
@grlee77 grlee77 force-pushed the skimage020-isotropic-binary-morphology branch from de983ab to 19080b3 Compare November 29, 2022 03:15
also test isotropic_opening and isotropic_closing with out kwarg
@grlee77
Copy link
Contributor Author

grlee77 commented Nov 29, 2022

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:

FAILED src/cucim/skimage/morphology/tests/test_isotropic.py::test_out_argument - 
cupy_backends.cuda.api.driver.CUDADriverError: CUDA_ERROR_ILLEGAL_ADDRESS: 
an illegal memory access was encountered

and then every subsequent test also fails with this same error.

@grlee77
Copy link
Contributor Author

grlee77 commented Nov 29, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit f5408d0 into rapidsai:branch-22.12 Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Introduces a non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants