Skip to content

Incorporate bug fixes from skimage 0.19.3 - #312

Merged
rapids-bot[bot] merged 4 commits into
rapidsai:branch-22.08from
grlee77:update-skimage-0.19.3
Jun 16, 2022
Merged

rapids-bot[bot] merged 4 commits into
rapidsai:branch-22.08from
grlee77:update-skimage-0.19.3

Conversation

@grlee77

@grlee77 grlee77 commented Jun 13, 2022

Copy link
Copy Markdown
Contributor

This PR incorporates the two bug fixes from skimage v0.19.3 (released on June 12th) that are relevant to cuCIM.

  • bug fix to ensure all color channel share the same histogram bins in cucim.skimage.exposure.histogram
  • clipping in warp functions should respect user-specified cval during clipping even if it is outside the original image range

There was also a fix to Canny edge detection, but we had already resolved that issue in cuCIM. I did go ahead and add the new test case for it to #310.

grlee77 added 2 commits June 13, 2022 10:34
This is an adaptation of the following PR included in skimage 0.19.3:
scikit-image/scikit-image#6335
@grlee77 grlee77 added bug Something isn't working non-breaking Introduces a non-breaking change labels Jun 13, 2022
@grlee77
grlee77 requested a review from a team as a code owner June 13, 2022 14:46
@jakirkham

Copy link
Copy Markdown
Member

rerun tests

Comment thread python/cucim/src/cucim/skimage/transform/tests/test_warps.py Outdated
Co-authored-by: jakirkham <jakirkham@gmail.com>
@grlee77

grlee77 commented Jun 16, 2022

Copy link
Copy Markdown
Contributor Author

Thanks for fixing the linting issue @jakirkham. I think we will still first need the bug fix in #310 in order for this PR to pass.

@jakirkham

Copy link
Copy Markdown
Member

rerun tests

@jakirkham

Copy link
Copy Markdown
Member

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit 3b4b9eb into rapidsai:branch-22.08 Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants