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

Allow numpy 1.24. #563

Merged
merged 2 commits into from
May 31, 2023
Merged

Allow numpy 1.24. #563

merged 2 commits into from
May 31, 2023

Conversation

bdice
Copy link
Contributor

@bdice bdice commented May 26, 2023

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
Copy link
Contributor

grlee77 commented May 31, 2023

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
Copy link
Contributor

grlee77 commented May 31, 2023

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

@bdice
Copy link
Contributor Author

bdice commented May 31, 2023

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
Member

/merge

@rapids-bot rapids-bot bot merged commit 87dd57e into rapidsai:branch-23.06 May 31, 2023
@jakirkham
Copy link
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