Summary
CI is failing on all Julia versions (LTS 1.10, 1.11, 1.12) due to issues in the upstream dependency GeometricIntegrators.jl.
Failure Analysis
Julia 1.10 (LTS)
TypeError: in Type{...} expression, expected UnionAll, got Type{SimpleSolvers.NonlinearSolver}
Location: ~/.julia/packages/GeometricIntegrators/Yo3w3/src/integrators/rk/integrators_firk.jl:65
Julia 1.11 & 1.12
ArgumentError: invalid type for argument sol in method definition for copy_solution!
Location: ~/.julia/packages/GeometricIntegrators/Yo3w3/src/solutions/atomic_solution.jl:32
Root Cause
The failures are in the upstream GeometricIntegrators.jl package (https://github.com/JuliaGNI/GeometricIntegrators.jl), not in this package. GeometricIntegratorsDiffEq.jl cannot function until GeometricIntegrators.jl is fixed for:
- Julia 1.10 - Type expression issue in
integrators_firk.jl
- Julia 1.11/1.12 - Method definition issue in
atomic_solution.jl
Suggested Actions
- Check if there's a newer version of
GeometricIntegrators.jl that addresses these issues
- Open an issue on the GeometricIntegrators.jl repository if not already reported
- Consider pinning to a working version of
GeometricIntegrators.jl if one exists
Reference
Discovered via CI health check PR: #43
cc @ChrisRackauckas
Summary
CI is failing on all Julia versions (LTS 1.10, 1.11, 1.12) due to issues in the upstream dependency
GeometricIntegrators.jl.Failure Analysis
Julia 1.10 (LTS)
Location:
~/.julia/packages/GeometricIntegrators/Yo3w3/src/integrators/rk/integrators_firk.jl:65Julia 1.11 & 1.12
Location:
~/.julia/packages/GeometricIntegrators/Yo3w3/src/solutions/atomic_solution.jl:32Root Cause
The failures are in the upstream
GeometricIntegrators.jlpackage (https://github.com/JuliaGNI/GeometricIntegrators.jl), not in this package.GeometricIntegratorsDiffEq.jlcannot function untilGeometricIntegrators.jlis fixed for:integrators_firk.jlatomic_solution.jlSuggested Actions
GeometricIntegrators.jlthat addresses these issuesGeometricIntegrators.jlif one existsReference
Discovered via CI health check PR: #43
cc @ChrisRackauckas