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

@juenglin juenglin 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-actions github-actions Bot added the cuda.core Everything related to the cuda.core module label Aug 5, 2026
@juenglin juenglin self-assigned this Aug 5, 2026
@juenglin juenglin added test Improvements or additions to tests P2 Low priority - Nice to have labels Aug 5, 2026
@juenglin juenglin 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:main Aug 7, 2026
111 of 116 checks passed
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
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

cuda.core Everything related to the cuda.core module P2 Low priority - Nice to have test Improvements or additions to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants