Skip to content

Conversation

@gmarkall
Copy link
Contributor

@gmarkall gmarkall commented May 2, 2025

Normal wheel tests should not install any CUDA pip packages because they should use the toolkit from the container.

The test depending on the toolkit wheels should install the specific toolkit version used for the test (as opposed to just using the latest version, which they were set up to do before).

@gmarkall gmarkall marked this pull request as draft May 2, 2025 17:10
@gmarkall gmarkall added the 2 - In Progress Currently a work in progress label May 2, 2025
Normal wheel tests should not install any CUDA pip packages because they
should use the toolkit from the container.

The test depending on the toolkit wheels should install the specific
toolkit version used for the test (as opposed to just using the latest
version, which they were set up to do before).
@gmarkall gmarkall force-pushed the fix-wheels-scripts branch from 9db56b1 to 1510c91 Compare May 3, 2025 09:52
@gmarkall gmarkall changed the title [CI TEST] Quick check to see if this fixes wheels CI Fix package installation for wheels CI May 3, 2025
@gmarkall gmarkall added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currently a work in progress labels May 3, 2025
@gmarkall gmarkall marked this pull request as ready for review May 3, 2025 09:57
Copy link
Contributor Author

@gmarkall gmarkall left a comment

Choose a reason for hiding this comment

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

As the CI is passing again, and test-wheel-deps-wheels is now getting the toolkit version specified from the environment, I'm going to merge this to proceed with the release.

@gmarkall gmarkall added 5 - Ready to merge Testing and reviews complete, ready to merge and removed 3 - Ready for Review Ready for review by team labels May 3, 2025
@gmarkall gmarkall merged commit ffc9c70 into NVIDIA:main May 3, 2025
37 checks passed
gmarkall added a commit to gmarkall/numba-cuda that referenced this pull request May 3, 2025
- Local variable debug info deduplication (NVIDIA#222)
- Fix package installation for wheels CI  (NVIDIA#238)
- Fix Invalid NVVM IR emitted when lowering shfl_sync APIs (NVIDIA#231)
- Add Bfloat16 Low++ Bindings (NVIDIA#166)
- Fix cuda.jit decorator inline (NVIDIA#181)
- Feature: cuda specific make_attribute_wrapper (NVIDIA#193)
- return a none tuple if no libdevice path is found (NVIDIA#234)
@gmarkall gmarkall mentioned this pull request May 3, 2025
gmarkall added a commit that referenced this pull request May 3, 2025
- Local variable debug info deduplication (#222)
- Fix package installation for wheels CI  (#238)
- Fix Invalid NVVM IR emitted when lowering shfl_sync APIs (#231)
- Add Bfloat16 Low++ Bindings (#166)
- Fix cuda.jit decorator inline (#181)
- Feature: cuda specific make_attribute_wrapper (#193)
- return a none tuple if no libdevice path is found (#234)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Testing and reviews complete, ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant