Skip to content

Allow numpy 1.24. - #563

Merged
rapids-bot[bot] merged 2 commits into
rapidsai:branch-23.06from
bdice:allow-numpy-1.24
May 31, 2023
Merged

rapids-bot[bot] merged 2 commits into
rapidsai:branch-23.06from
bdice:allow-numpy-1.24

Conversation

@bdice

@bdice bdice commented May 26, 2023

Copy link
Copy Markdown
Contributor

This removes the upper bound on numpy<1.24 to allow newer NumPy versions.

@bdice
bdice requested a review from a team as a code owner May 26, 2023 21:28
@jakirkham jakirkham added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels May 27, 2023
@grlee77

grlee77 commented May 31, 2023

Copy link
Copy Markdown
Contributor

The SHA256 failures here are related to a recent change in the eagle source image: https://gitlab.com/scikit-image/data/-/commit/1e4f62ac31ba4553d176d4473a5967ad1b076d62

skimage 0.19 seems to not handle this properly. It downloads the new version, but compares it with the old hash value.

skimage 0.20 downloads the older version so the old hash value check passes.

for upcoming skimage 0.21 it will download the new version and pass with the new hash value.

Two possible workarounds for cuCIM are:
1.) skip the test case using the eagle image when scikit-image.version < 0.20
2.) make CI use scikit-image>=0.20

@grlee77

grlee77 commented May 31, 2023

Copy link
Copy Markdown
Contributor

Thanks @bdice, it looks good. Opened #564 to resolve the unrelated test failures

@bdice

bdice commented May 31, 2023

Copy link
Copy Markdown
Contributor Author

Great. Once #564 is merged, feel free to update this PR with branch-23.06 and merge.

rapids-bot Bot pushed a commit that referenced this pull request May 31, 2023
Due to a [recent change in the eagle image](https://gitlab.com/scikit-image/data/-/merge_requests/22) used in one of our test cases, skimage 0.19 fails to load it. As a solution, we can just restrict this one test to run only for `scikit-image>=0.20`.

Should resolve test failures seen in #563

Authors:
  - Gregory Lee (https://github.com/grlee77)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - https://github.com/jakirkham

URL: #564
@jakirkham

Copy link
Copy Markdown
Member

/merge

@rapids-bot
rapids-bot Bot merged commit 87dd57e into rapidsai:branch-23.06 May 31, 2023
@jakirkham

Copy link
Copy Markdown
Member

Thanks all! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants