Fix test warnings: suppress expected GPU/NaN warnings, fix real code issues#887
Merged
brendancol merged 1 commit intomasterfrom Feb 24, 2026
Merged
Fix test warnings: suppress expected GPU/NaN warnings, fix real code issues#887brendancol merged 1 commit intomasterfrom
brendancol merged 1 commit intomasterfrom
Conversation
…version and host-array copies - Add [tool:pytest] filterwarnings to silence NumbaPerformanceWarning from small test arrays - Suppress RuntimeWarning for all-NaN slices in _run_equal_interval (valid code path) - Fix DeprecationWarning in curvature.py by extracting scalar explicitly via cupy.asnumpy().item() - Convert kernel to device array in focal.py and convolution.py to avoid implicit host-to-device copies
This was referenced Feb 24, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
[tool:pytest]filterwarnings insetup.cfgto silenceNumbaPerformanceWarning(GPU under-utilization from small test arrays — 28 instances)RuntimeWarningfor all-NaN slices inclassify.py:_run_equal_interval, which is a valid code path handled correctly downstream (2 instances)DeprecationWarningincurvature.pyby extracting scalars explicitly viacupy.asnumpy().item()instead offloat()on an ndarray (12 instances)focal.pyandconvolution.pyby converting kernel tocupy.asarray()before passing to CUDA kernels (10 instances)Reduces test warnings from 55 to 0 under normal
pytestruns.Test plan
pytest --tb=short -qpasses (843 passed, 12 skipped, 0 warnings)pytest -W allconfirms DeprecationWarning, "Host array", and "All-NaN slice" warnings are gone (only expected NumbaPerformanceWarning remains, which-W allforces visible)