Skip to content

Gracefully skip CUDA tests when CUDAdrv is unavailable#27

Merged
kdw503 merged 1 commit intomasterfrom
dwk/fix-cuda-test
May 8, 2026
Merged

Gracefully skip CUDA tests when CUDAdrv is unavailable#27
kdw503 merged 1 commit intomasterfrom
dwk/fix-cuda-test

Conversation

@kdw503
Copy link
Copy Markdown
Member

@kdw503 kdw503 commented May 8, 2026

Summary

  • Wraps the CUDAdrv/RegisterMismatchCuda test setup in a try/catch so the suite falls back to CPU-only when CUDAdrv is not installed
  • CUDAdrv is a legacy package that is absent even when CUDA runtime libraries are present on the system, which caused the entire test suite to fail

Test plan

  • CPU-only tests pass locally without CUDAdrv installed

🤖 Generated with Claude Code

CUDAdrv is a legacy package that may not be installed even when CUDA
libraries are present on the system. Wrap the CUDA setup in a try/catch
so the test suite falls back to CPU-only rather than erroring out.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.02%. Comparing base (36a5ae5) to head (1b2d752).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   87.02%   87.02%           
=======================================
  Files           1        1           
  Lines         131      131           
=======================================
  Hits          114      114           
  Misses         17       17           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kdw503 kdw503 merged commit 899d697 into master May 8, 2026
4 checks passed
@kdw503 kdw503 deleted the dwk/fix-cuda-test branch May 8, 2026 19:12
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.

1 participant