Move interface tests to Differentiation and NonlinearSolve subpackages#3156
Merged
ChrisRackauckas merged 1 commit intoSciML:masterfrom Mar 17, 2026
Merged
Conversation
…fEqNonlinearSolve Move tests from the global test suite to their respective subpackages so they only run when the relevant subpackage changes, reducing CI cost. Tests moved to OrdinaryDiffEqDifferentiation: - differentiation_traits_tests.jl (from InterfaceI) - autodiff_error_tests.jl (from InterfaceIV) - nojac_tests.jl (from InterfaceIII) Tests moved to OrdinaryDiffEqNonlinearSolve: - linear_nonlinear_tests.jl (from InterfaceII) - linear_solver_tests.jl (from InterfaceII) - linear_solver_split_ode_tests.jl (from InterfaceII) - mass_matrix_tests.jl (from InterfaceII) - wprototype_tests.jl (from InterfaceII) - dae_initialization_tests.jl (from InterfaceV) - checkinit_tests.jl (from InterfaceII) Tests kept in global suite (require special envs): - autosparse_detection_tests.jl (SparseConnectivityTracer dep conflict) - gpu_autodiff_interface_tests.jl (GPU env) - nonfulldiagonal_sparse.jl (ComponentArrays dep) - AD tests (Enzyme/Zygote/Mooncake env) - ModelingToolkit tests (downstream dep) Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Member
|
Why did this PR add rather than move the tests? Shouldn't they be removed from the |
Member
|
Oops that's my bad, I made sure they were removed from Runtests but orphaned them 😅 will fix. |
ChrisRackauckas-Claude
pushed a commit
to ChrisRackauckas-Claude/OrdinaryDiffEq.jl
that referenced
this pull request
Mar 18, 2026
These 7 test files in test/interface/ were copied to OrdinaryDiffEqDifferentiation and OrdinaryDiffEqNonlinearSolve subpackages in PR SciML#3156 but the originals were never removed. They are no longer referenced by test/runtests.jl. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2 tasks
ChrisRackauckas
added a commit
that referenced
this pull request
Mar 18, 2026
…interface-tests Delete orphaned test files moved to subpackages in #3156
singhharsh1708
pushed a commit
to singhharsh1708/OrdinaryDiffEq.jl
that referenced
this pull request
Mar 25, 2026
These 7 test files in test/interface/ were copied to OrdinaryDiffEqDifferentiation and OrdinaryDiffEqNonlinearSolve subpackages in PR SciML#3156 but the originals were never removed. They are no longer referenced by test/runtests.jl. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
singhharsh1708
pushed a commit
to singhharsh1708/OrdinaryDiffEq.jl
that referenced
this pull request
Mar 30, 2026
These 7 test files in test/interface/ were copied to OrdinaryDiffEqDifferentiation and OrdinaryDiffEqNonlinearSolve subpackages in PR SciML#3156 but the originals were never removed. They are no longer referenced by test/runtests.jl. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
3 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
OrdinaryDiffEqDifferentiationandOrdinaryDiffEqNonlinearSolvesubpackagesTests moved to OrdinaryDiffEqDifferentiation (3 files):
differentiation_traits_tests.jl— jacobian/tgrad trait dispatch, AD backend selectionautodiff_error_tests.jl— error handling when AD fails (FirstAutodiffJacError, etc.)nojac_tests.jl— verifying no jacobian config allocated with Krylov solversTests moved to OrdinaryDiffEqNonlinearSolve (7 files):
linear_nonlinear_tests.jl— preconditioner interaction with nonlinear solverslinear_solver_tests.jl— KLU/UMFPACK/Krylov linear solver selectionlinear_solver_split_ode_tests.jl— SplitODEProblem with custom linsolvemass_matrix_tests.jl— mass matrices with NLFunctional/NLAnderson/NLNewtonwprototype_tests.jl— W matrix prototypes with SciMLOperatorsdae_initialization_tests.jl— ShampineCollocationInit/BrownFullBasicInitcheckinit_tests.jl— CheckInit validation for DAEsTests kept in global suite (require special environments):
autosparse_detection_tests.jl— SparseConnectivityTracer dep conflict with PreallocationToolsgpu_autodiff_interface_tests.jl— requires GPU env (JLArrays)nonfulldiagonal_sparse.jl— requires ComponentArraysTest plan
🤖 Generated with Claude Code