Skip to content

Fix CI: Exclude Enzyme tests from Julia pre#36

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:fix-ci-20251229
Dec 29, 2025
Merged

Fix CI: Exclude Enzyme tests from Julia pre#36
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:fix-ci-20251229

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR fixes CI by splitting tests into separate groups and excluding Enzyme tests from Julia pre-release versions.

The Julia 1.13-alpha tests fail due to an LLVM data layout incompatibility in Enzyme:

LLVM error: Added modules have incompatible data layouts: 
e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128 (module) vs 
e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128-ni:10:11:12:13 (jit)

This is an upstream Enzyme issue tracked at EnzymeAD/Enzyme.jl#2699.

Changes

  1. Split tests into groups: Modified test/runtests.jl to support Core and Enzyme test groups via the GROUP environment variable
  2. Matrix exclude: Updated Tests.yml to use matrix exclude pattern (similar to OrdinaryDiffEq) to skip Enzyme tests on Julia pre-release

Test Matrix

Julia Version Core Enzyme
1 (stable)
lts
pre ❌ (excluded)

Test Plan

  • Core tests pass locally
  • Enzyme tests pass locally
  • CI should pass for all included matrix combinations

cc @ChrisRackauckas

🤖 Generated with Claude Code

Split tests into Core and Enzyme groups, with Enzyme excluded from Julia
pre-release versions due to upstream LLVM data layout incompatibility.

See EnzymeAD/Enzyme.jl#2699 for the upstream issue.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ChrisRackauckas-Claude ChrisRackauckas-Claude changed the title Fix CI: Format EnzymeExt and allow Julia pre failures Fix CI: Exclude Enzyme tests from Julia pre Dec 29, 2025
@ChrisRackauckas ChrisRackauckas merged commit b9b5c58 into SciML:main Dec 29, 2025
16 of 17 checks passed
@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor Author

Added fix for formatting issue in test/runtests.jl - missing spaces around the operator.

cc @ChrisRackauckas

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