Skip to content

chore: avoid some warnings when running cuda.core tests - #2515

Merged
juenglin merged 1 commit into
NVIDIA:mainfrom
juenglin:pytest-warnings
Aug 7, 2026
Merged

chore: avoid some warnings when running cuda.core tests#2515
juenglin merged 1 commit into
NVIDIA:mainfrom
juenglin:pytest-warnings

Conversation

@juenglin

@juenglinjuenglin commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Fix pytest warnings in cuda_core test suite

Register the thread_unsafe and parallel_threads_limit custom marks in
pytest.ini to eliminate PytestUnknownMarkWarning noise across the suite.

Wrap itertools.combinations(...) arguments in list() in
test_object_protocols.py to fix PytestRemovedIn10Warning about passing
lazy iterators to @pytest.mark.parametrize (deprecated ahead of pytest 10).

Add @pytest.mark.filterwarnings("ignore::DeprecationWarning") to
test_temperature in test_system_device.py. The test exercises
get_threshold for all threshold types, including the four that
intentionally emit a DeprecationWarning on Ada+ hardware. The warning
behaviour itself is covered by the dedicated
test_temperature_threshold_unrecognized_device_arch test.

@github-actionsgithub-actionsBot added the cuda.core Everything related to the cuda.core module label Aug 5, 2026
@juenglinjuenglin self-assigned this Aug 5, 2026
@juenglinjuenglin added test Improvements or additions to tests P2 Low priority - Nice to have labels Aug 5, 2026
@juenglinjuenglin added this to the cuda.core 1.2.0 milestone Aug 5, 2026
@github-actions

This comment has been minimized.

@juenglin
juenglin merged commit be1e752 into NVIDIA:mainAug 7, 2026
111 of 116 checks passed
@github-actions

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

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

Labels

cuda.coreEverything related to the cuda.core moduleP2Low priority - Nice to havetestImprovements or additions to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@juenglin@mdboom