Skip to content

ci: fix compilation on self-hosted CI and test with multiple versions of CUDA#3704

Merged
ariostas merged 4 commits intoscikit-hep:mainfrom
ariostas:fix_gpu_ci
Oct 28, 2025
Merged

ci: fix compilation on self-hosted CI and test with multiple versions of CUDA#3704
ariostas merged 4 commits intoscikit-hep:mainfrom
ariostas:fix_gpu_ci

Conversation

@ariostas
Copy link
Copy Markdown
Member

Seems like there were some issues after I switched the GPU CI to a different Docker image. Sorry about that. I'm fixing those issues here, and I'll also see if I can get other versions of CUDA working as well.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.72%. Comparing base (b749e49) to head (f8f4d50).
⚠️ Report is 458 commits behind head on main.

Additional details and impacted files

see 200 files with indirect coverage changes

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

@github-actions
Copy link
Copy Markdown

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR3704

@ariostas ariostas changed the title ci: fix GPU CI ci: fix compilation on self-hosted CI and test with multiple versions of CUDA Oct 28, 2025
@ariostas
Copy link
Copy Markdown
Member Author

The wheel building works now, and I also got it to work with CUDA 11 and 13, in addition to CUDA 12.

@ariostas ariostas marked this pull request as ready for review October 28, 2025 15:30
@ariostas ariostas requested a review from ianna October 28, 2025 15:32
Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

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

@ariostas - Great! Thanks for making this work. Please merge it if you are done with it. Thanks.

@ariostas ariostas merged commit 7849d51 into scikit-hep:main Oct 28, 2025
44 checks passed
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