Skip to content

Conversation

@rwgk
Copy link
Collaborator

@rwgk rwgk commented Jan 12, 2026

Move supports_ipc_mempool from cuda_python_test_helpers to cuda_core/tests/helpers/__init__.py to remove the dependency on cuda_python_test_helpers for this cuda-core-specific functionality.

This makes cuda_python_test_helpers independent of cuda-core-specific packages, allowing it to be used in other contexts without requiring cuda-core or cuda-bindings to be installed.

Changes:

  • Add supports_ipc_mempool to cuda_core/tests/helpers/__init__.py
  • Update cuda_core/tests/test_memory.py to import from helpers
  • Remove supports_ipc_mempool from cuda_python_test_helpers
  • Remove unused imports (functools, Union, handle_return) from cuda_python_test_helpers

Note: This PR was extracted from PR #1454

Move supports_ipc_mempool from cuda_python_test_helpers to
cuda_core/tests/helpers/__init__.py to remove the dependency on
cuda_python_test_helpers for this CUDA-specific functionality.

This makes cuda_python_test_helpers independent of CUDA-specific
packages, allowing it to be used in other contexts without requiring
cuda-core or cuda-bindings to be installed.

Changes:
- Add supports_ipc_mempool to cuda_core/tests/helpers/__init__.py
- Update cuda_core/tests/test_memory.py to import from helpers
- Remove supports_ipc_mempool from cuda_python_test_helpers
- Remove unused imports (functools, Union, handle_return) from
  cuda_python_test_helpers
@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 12, 2026

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 changed the title Move supports_ipc_mempool to cuda_core-specific helpers Move supports_ipc_mempool to cuda_core-specific helpers Jan 12, 2026
@rwgk
Copy link
Collaborator Author

rwgk commented Jan 12, 2026

/ok to test

@github-actions

This comment has been minimized.

Copy link
Contributor

@Andy-Jost Andy-Jost left a comment

Choose a reason for hiding this comment

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

LGTM

@rwgk rwgk merged commit 6afdd5c into NVIDIA:main Jan 12, 2026
80 checks passed
@rwgk rwgk deleted the test_helper_supports_ipc_mempool_factoring branch January 12, 2026 22:10
@github-actions
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants