-
Notifications
You must be signed in to change notification settings - Fork 59
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
[BUG] some thresholding functions require scikit-image>=0.19 #363
Comments
It looks like we didn't bump this dependency. Should we be doing this? Also is this only test related or would users see this in workflows? cucim/conda/recipes/cucim/meta.yaml Line 35 in e155e32
|
The functions failing with 0.18 are;
All will work if the user upgrades to >=0.19. |
This commit where we deferred some computations to scikit-image and removed duplicate code is what lead to the issue. |
this will avoid the scikit-image 0.18.x issues with 3 thresholding functions observed in rapidsaigh-363: rapidsai#363
Describe the bug
Changes made in #276 break use of some thresholding functions if the user does not have scikit-image >= 0.19. We should either bump our minimum requirement to 0.19 or provide a fallback.
Steps/Code to reproduce bug
many test failures are seen for
cucim/skimage/filters/tests/test_thresholding.py
Expected behavior
all tests should pass
Environment details (please complete the following information):
cuCIM branch 22.08
The text was updated successfully, but these errors were encountered: