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

Build with NumPy 1.17 #148

Merged
merged 2 commits into from
Nov 19, 2021
Merged

Conversation

jakirkham
Copy link
Member

Instead of letting host's numpy have a minimum, which could float to a high version that is in turn pinned to a high version in run, build using the minimum required numpy version. That way the minimum is set similarly in host and a wide range of numpy's will still work at runtime.

cc @quasiben (for awareness)

Instead of letting `host`'s `numpy` have a minimum, which could float to
a high version that is in turn pinned to a high version in `run`, build
using the minimum required `numpy` version. That way the minimum is set
similarly in `host` and a wide range of `numpy`'s will still work at
runtime.
@jakirkham jakirkham requested a review from a team as a code owner November 16, 2021 19:22
@jakirkham jakirkham added bug Something isn't working non-breaking Introduces a non-breaking change labels Nov 16, 2021
@jakirkham
Copy link
Member Author

jakirkham commented Nov 16, 2021

In particular this caused some issues when installing numba into the same environment as it doesn't always support the latest numpy, but cucim was effectively being built with and requiring the latest numpy. So both cucim and numba couldn't be installed in the same environment. This should fix that.

Seems to be overly strict from a solver perspective.
@jakirkham
Copy link
Member Author

rerun tests

(one of the jobs failed, but it looks unrelated)

@jakirkham
Copy link
Member Author

rerun tests

(pending job was likely lost during rename)

@jakirkham
Copy link
Member Author

@gpucibot merge

@quasiben
Copy link
Member

rerun tests

@jakirkham
Copy link
Member Author

Debugging in PR ( #153 )

@ajschmidt8
Copy link
Member

the segfault failure in this PR is unrelated to the numpy version change. I will admin merge this PR to unblock some other conda solve issues downstream that this PR resolves.

@ajschmidt8 ajschmidt8 merged commit 3af952f into rapidsai:branch-21.12 Nov 19, 2021
@jakirkham jakirkham deleted the fix_np_hst branch November 19, 2021 16:59
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.

4 participants