Skip to content

Build with NumPy 1.17 - #148

Merged
ajschmidt8 merged 2 commits into
rapidsai:branch-21.12from
jakirkham:fix_np_hst
Nov 19, 2021
Merged

ajschmidt8 merged 2 commits into
rapidsai:branch-21.12from
jakirkham:fix_np_hst

Conversation

@jakirkham

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

jakirkham commented Nov 16, 2021 •

Copy link
Copy Markdown
Member Author

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
Copy Markdown
Member Author

rerun tests

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

@jakirkham

Copy link
Copy Markdown
Member Author

rerun tests

(pending job was likely lost during rename)

@jakirkham

Copy link
Copy Markdown
Member Author

@gpucibot merge

@quasiben

Copy link
Copy Markdown
Member

rerun tests

@jakirkham

Copy link
Copy Markdown
Member Author

Debugging in PR ( #153 )

@ajschmidt8

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