Skip to content

Don't constrain numba<0.58 - #14616

Merged
rapids-bot[bot] merged 7 commits into
NVIDIA:branch-24.02from
brandon-b-miller:numba-058
Dec 18, 2023
Merged

Don't constrain numba<0.58#14616
rapids-bot[bot] merged 7 commits into
NVIDIA:branch-24.02from
brandon-b-miller:numba-058

Conversation

@brandon-b-miller

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

Copy link
Copy Markdown
Contributor

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

bdice commented Dec 12, 2023

Copy link
Copy Markdown
Contributor

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

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
Comment thread dependencies.yaml Outdated
Comment thread conda/environments/all_cuda-120_arch-x86_64.yaml Outdated
Comment thread dependencies.yaml Outdated
@jakirkham

Copy link
Copy Markdown
Contributor

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

/merge

@rapids-bot
rapids-bot Bot merged commit 8dca25c into NVIDIA:branch-24.02 Dec 18, 2023
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