Fix CUDA tests and docs build#198
Open
ChrisRackauckas-Claude wants to merge 1 commit intoSciML:mainfrom
Open
Conversation
Changes: 1. Fix runtests.jl to use BACKEND_GROUP instead of GROUP env var - The GPU.yml workflow sets BACKEND_GROUP=CUDA but tests were looking for GROUP - Now tests properly run when BACKEND_GROUP=CUDA is set 2. Add LuxCUDA to test dependencies - shared_testsetup.jl tries to 'using LuxCUDA' when running CUDA tests - LuxCUDA was missing from test deps causing LoadError 3. Add LocalPreferences.toml for docs build (V100 compatibility) - Pin CUDA runtime to 12.6 and disable forward-compat driver - Fixes demeter4 V100 runners where CUDA_Driver_jll v13+ drops CC 7.0 support - Add CUDA_Driver_jll and CUDA_Runtime_jll to docs deps Fixes: ChrisRackauckas/InternalJunk#22
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
2 participants
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
Fixes the CUDA test and documentation build failures from the GHA migration (PR #197).
Changes
1. Fix runtests.jl to use BACKEND_GROUP instead of GROUP
The GPU.yml workflow sets
BACKEND_GROUP=CUDAbut the tests were looking forGROUPenv var. This caused the tests to default to CPU mode and fail when trying to load LuxCUDA (which wasn't needed for CPU tests).2. Add LuxCUDA to test dependencies
shared_testsetup.jltries tousing LuxCUDAwhen running CUDA tests, but LuxCUDA was missing from the test dependencies in Project.toml. This caused the LoadError:3. Add LocalPreferences.toml for docs build (V100 compatibility)
The documentation build failed on demeter4 (V100 GPU runners) due to CUDA version incompatibility. Following the pattern from OrdinaryDiffEq.jl and the fix documented in ChrisRackauckas/InternalJunk#19:
Related Issues
Fixes: ChrisRackauckas/InternalJunk#22