Move Enzyme tests to NoPre test group to fix prerelease CI failures #672
Conversation
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
This change addresses CI failures on Julia prerelease versions by: - Creating a new 'nopre' test group for tests that use Enzyme - Adding the nopre group to CI configuration for NonlinearSolve and SimpleNonlinearSolve - Excluding nopre tests from running on Julia 'pre' version in CI - Creating enzyme_tests.jl with example Enzyme tests that skip on prerelease All existing Enzyme tests in subpackages already have VERSION.prerelease checks, but this change provides an additional CI-level control to prevent these tests from running on prerelease Julia versions. Fixes test failures seen in https://github.com/SciML/NonlinearSolve.jl/actions/runs/16851672503/job/47738829623 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Instead of creating new tests, this moves existing tests that could potentially use Enzyme through default autodiff settings to the nopre group: - Basic PolyAlgorithms: Uses RobustMultiNewton() and FastShortcutNonlinearPolyalg() - PolyAlgorithms Autodiff: Tests autodiff with polyalgorithms - 23 Test Problems: PolyAlgorithms: Tests with default polyalgorithms - ForwardDiff.jl Integration: Tests with FastShortcutNonlinearPolyalg() and nothing - Ensemble Nonlinear Problems: Uses nothing as algorithm - Out-of-place Matrix Resizing: Uses polyalgorithms without explicit autodiff - Inplace Matrix Resizing: Uses polyalgorithms without explicit autodiff - Default Algorithm Singular Handling: Uses default solver These tests may trigger Enzyme usage through default autodiff backends, so moving them to nopre group ensures they don't run on Julia prerelease. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Update: Moved existing tests to nopre groupAs requested, I've removed the new test file and instead moved existing tests that could potentially use Enzyme through default autodiff settings to the Tests moved from
|
- Remove Enzyme from regular test dependencies in Project.toml - Add Enzyme as an EXTRA_PKG that's loaded conditionally in runtests.jl - Only load Enzyme for the nopre group when not on prerelease Julia - This ensures Enzyme is only loaded when needed and won't cause issues on prerelease This complements the previous change of moving tests to the nopre group, ensuring Enzyme is truly optional and only loaded when appropriate. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
nopretest group for tests that use Enzymenopretests from running on Julia prerelease versions in CIProblem
The CI is currently failing on Julia prerelease versions due to Enzyme compatibility issues, as seen in https://github.com/SciML/NonlinearSolve.jl/actions/runs/16851672503/job/47738829623
While all existing Enzyme tests in subpackages already have
VERSION.prereleasechecks to skip on prerelease Julia, having a dedicated test group provides better CI-level control.Solution
noprethat can contain tests that should not run on Julia prereleasenopregroup to the CI matrix for NonlinearSolve and SimpleNonlinearSolvenopretests from running whenversion: "pre"test/enzyme_tests.jlwith example Enzyme tests that demonstrate the patternTest plan
🤖 Generated with Claude Code