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

Add numba to nightly upstream #7311

Closed
wants to merge 2 commits into from
Closed

Conversation

max-sixty
Copy link
Collaborator

I saw that we didn't have this while investigating numba/numba#8615. We should probably wait until that's resolved before merging this (this doesn't solve that issue).

I saw that we didn't have this while investigating numba/numba#8615. We should probably wait until that's resolved before merging this (this doesn't solve that issue).
@github-actions github-actions bot added the CI Continuous Integration tools label Nov 22, 2022
h5netcdf \
matplotlib \
numpy \
numba \
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only semantic change; the others are just sorting the lines

git+https://github.com/hgrecco/pint \
git+https://github.com/numba/numba \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does numba require compilation? If so, it might be better to use pre-built packages (if numba provides those?).

@dcherian
Copy link
Contributor

dcherian commented Dec 1, 2022

Since numba won't support numpy HEAD in general, shall we remove it from the env instead?

@max-sixty
Copy link
Collaborator Author

Since numba won't support numpy HEAD in general, shall we remove it from the env instead?

Yes, I'll get to it if no one gets there first. I guess we also need to disable those tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration tools upstream issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants