Skip to content

Restore option to run testing without cupy installed. #741

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Jul 1, 2025

Description

cuda_core is generally set up to run most tests without cupy installed.

PR #717 added one unconditional cupy import. This PR adopts the approach in tests/test_utils.py for the import in tests/test_launcher.py, and copies the SKIPPED message from tests/example_tests/utils.py.

Copy link
Contributor

copy-pr-bot bot commented Jul 1, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk
Copy link
Collaborator Author

rwgk commented Jul 1, 2025

/ok to test

This comment has been minimized.

@rwgk rwgk marked this pull request as ready for review July 1, 2025 07:24
Copy link
Contributor

copy-pr-bot bot commented Jul 1, 2025

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk rwgk requested a review from shwina July 1, 2025 07:25
@github-project-automation github-project-automation bot moved this from Todo to In Review in CCCL Jul 1, 2025
@leofang leofang added P1 Medium priority - Should do test Improvements or additions to tests cuda.core Everything related to the cuda.core module labels Jul 1, 2025
@rwgk
Copy link
Collaborator Author

rwgk commented Jul 1, 2025

Thanks @leofang

@rwgk rwgk merged commit 5576da6 into NVIDIA:main Jul 1, 2025
56 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Jul 1, 2025
@rwgk rwgk deleted the cuda_core_cupy_skip branch July 1, 2025 11:31
Copy link

github-actions bot commented Jul 1, 2025

Doc Preview CI
Preview removed because the pull request was closed or merged.

@leofang leofang added this to the cuda.core beta 5 milestone Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda.core Everything related to the cuda.core module P1 Medium priority - Should do test Improvements or additions to tests
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants