Skip to content

Implement additional deprecations carried out for scikit-image 0.20 - #451

Merged
rapids-bot[bot] merged 7 commits into
rapidsai:branch-22.12from
grlee77:skimage020-deprecations
Nov 30, 2022
Merged

rapids-bot[bot] merged 7 commits into
rapidsai:branch-22.12from
grlee77:skimage020-deprecations

Conversation

@grlee77

@grlee77 grlee77 commented Nov 18, 2022

Copy link
Copy Markdown
Contributor

related to #419

This PR removes most of the deprecated parameters that are being removed in scikit-image 0.20.

There are a few exceptions (e.g. selem) where they will be removed in scikit-image 0.20, but we had previously advertised a specific removal date like 2022.03.02. For those, I will delay the removal until the previously advertised release.

It will be nice to be rid of all the deprecated multichannel code! The first, large commit here is just the multichannel deprecation. The rest of the commits are much smaller.

skimage 0.20 will also remove selem and a few other arguments, but we had previously
scheduled those for removal in cucim 2023.02.00, so will stick to that schedule

fix version string comparison in existing _denoise_wavelet utility in test_j_invariant.py
@grlee77 grlee77 added breaking Introduces a breaking change improvement Improves an existing functionality labels Nov 18, 2022
@grlee77 grlee77 added this to the v22.12.00 milestone Nov 18, 2022
@grlee77
grlee77 requested a review from a team as a code owner November 18, 2022 20:00
@grlee77

grlee77 commented Nov 29, 2022

Copy link
Copy Markdown
Contributor Author

These deprecations have been approved, but not yet merged upstream. See comment here: scikit-image/scikit-image#6583 (comment)

I think it would be nice to not have all of these deprecated parameters around, but am also okay if we want to bump this until 2022.02.00.

@gigony gigony left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @grlee77 ! It looks good to me!

@grlee77

grlee77 commented Nov 30, 2022

Copy link
Copy Markdown
Contributor Author

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit a7490ca into rapidsai:branch-22.12 Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change improvement Improves an existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants