[mdboom]

0271e3a broke local cuda_core testing.

To reproduce:

python -m pip install ./cuda_bindings_13
python -m pip install ./cuda_core
python -m pytest cuda_core/tests

Pytest resolves those filterwarnings entries while loading config, which is before cuda_core/tests/conftest.py runs. At that point it imports helpers, and pythonpath = tests makes that importable.
The conftest's sys.path fallback for cuda_python_test_helpers hasn't run yet. From the repo root, cuda_python_test_helpers resolves to the empty namespace directory.
helpers/init.py does from cuda_python_test_helpers import *, which binds nothing and raises nothing. So helpers finishes importing and stays cached in sys.modules without IS_WINDOWS, IS_WSL or libc.
helpers.memory then fails to import. That failure is the "Failed to import filter module" warning the pytest.ini comment says it ignores. The cached helpers package stays, and the conftest bootstrap never evicts it.
Every later from helpers... import IS_WINDOWS then fails.

Avoiding import * is the simplest way to fix this.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: NVIDIA/cuda-python
• Branch: main
• File: .coderabbit.yaml
• Commit: fd5f89d
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 50d9b07f-ffc7-40c3-9475-05a42391de4d

Commits

Reviewing files that changed from the base of the PR and between fd5f89d and 04f87e9.

Files selected for processing: 1

• cuda_core/tests/helpers/__init__.py

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

• Chores
  • Made a minor internal maintenance change. There are no user-visible changes or changes to application behavior.

The test helpers package replaces a wildcard import with explicit imports of five names from cuda_python_test_helpers.

Changes: Test helper imports

Layer / File(s)	Summary
Explicit helper imports cuda_core/tests/helpers/__init__.py	The initializer explicitly imports IS_LINUX, IS_WINDOWS, IS_WSL, libc, and under_compute_sanitizer.

Possibly related issues

• CI doesn't test the latest release of cuda_core against cuda_bindings main #2449 — The import change is related to the reported local testing setup with installed cuda_bindings_13 and cuda_core.

Possibly related PRs

• NVIDIA/cuda-python#3173 — It addresses the repository-local test helper import path and module cache behavior.

Suggested labels: cuda.core, test

Suggested reviewers: cpcloud

Priority: Normal

Change: Bug fix

Merge Risk: Minimal · up to 04f87

No merge-blocking risk was identified in this focused test-helper import change.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

Finishing Touches: 💡 1

🛠️ Fix failing CI checks: 💡

• Commit to this branch
• Create a new PR

[github-actions]

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

[juenglin]

Thank you.

[leofang]

CI failed due to #3038. I re-ran it.