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

Don't constrain numba<0.58 #14616

Merged
merged 7 commits into from
Dec 18, 2023
Merged

Conversation

brandon-b-miller
Copy link
Contributor

@brandon-b-miller brandon-b-miller commented Dec 12, 2023

This PR removes the constraint for numba 0.58 in our dependencies.

Closes #14160

@brandon-b-miller brandon-b-miller requested review from a team as code owners December 12, 2023 18:01
@github-actions github-actions bot added Python Affects Python cuDF API. conda labels Dec 12, 2023
@brandon-b-miller brandon-b-miller added feature request New feature or request non-breaking Non-breaking change Python Affects Python cuDF API. numba Numba issue and removed Python Affects Python cuDF API. conda labels Dec 12, 2023
@bdice
Copy link
Contributor

bdice commented Dec 12, 2023

Should we test with development builds of numba 0.59 to ensure we don't see breakage from that release? I'm happy either way, just wanted to pose the question. cc: @gmarkall

@jakirkham
Copy link
Member

Should we test with development builds of numba 0.59 to ensure we don't see breakage from that release?

Could try adding their dev channel numba/label/ci and seeing what happens. Would make sure it is installing the latest numba. Have noticed that doesn't always work out-of-the-box with the dev channel

@github-actions github-actions bot added the conda label Dec 14, 2023
dependencies.yaml Outdated Show resolved Hide resolved
conda/environments/all_cuda-120_arch-x86_64.yaml Outdated Show resolved Hide resolved
dependencies.yaml Outdated Show resolved Hide resolved
@jakirkham
Copy link
Member

For clarity don't think the test actually worked: #14616 (comment)

@github-actions github-actions bot added the ci label Dec 14, 2023
@github-actions github-actions bot removed the ci label Dec 15, 2023
@brandon-b-miller
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 8dca25c into rapidsai:branch-24.02 Dec 18, 2023
67 checks passed
raydouglass added a commit that referenced this pull request Feb 13, 2024
I think unpinning `numba` in the conda recipe was just missed in #14616.

I discovered this issue [trying to build the `24.02` release](https://github.com/rapidsai/cudf/actions/runs/7878153691/job/21496377912#step:7:1674).

PRs & nightly builds are working because the `rapidsai-nightly` channel has an older version of `pynvjitlink` that supported `numba>=0.57` whereas the `rapidsai` channel only has the latest version which pins to `numba>=0.58`.

Authors:
   - Ray Douglass (https://github.com/raydouglass)

Approvers:
   - Vyas Ramasubramani (https://github.com/vyasr)
   - Bradley Dice (https://github.com/bdice)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Non-breaking change numba Numba issue Python Affects Python cuDF API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Errors and deprecation warnings from numba 0.58
6 participants