Skip to content

Move Enzyme tests to separate test group#46

Merged
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:enzyme-test-group
Jan 13, 2026
Merged

Move Enzyme tests to separate test group#46
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:enzyme-test-group

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor

Summary

  • Created test/enzyme/ directory with its own Project.toml for Enzyme dependencies
  • Moved all Enzyme-specific tests to test/enzyme/runtests.jl
  • Removed Enzyme from main test dependencies in Project.toml
  • Added separate CI job that only runs Enzyme tests on Julia LTS version

This follows the OrdinaryDiffEq.jl pattern for test groups with separate Project.toml files, which:

  1. Isolates Enzyme dependencies from the main test suite
  2. Avoids Enzyme compatibility issues with pre-release Julia versions (EnzymeRuntimeActivityError on Julia 1.12+)
  3. Keeps the main test suite running on all Julia versions (lts, 1, pre)

Test plan

  • CI passes for main tests on all Julia versions (lts, 1, pre)
  • CI passes for Enzyme tests on Julia LTS
  • Verify Enzyme tests cover the same functionality as before

🤖 Generated with Claude Code

This follows the OrdinaryDiffEq.jl pattern for test groups:
- Create test/enzyme/ directory with its own Project.toml
- Move all Enzyme-specific tests to test/enzyme/runtests.jl
- Remove Enzyme from main test dependencies
- Add separate CI job that only runs Enzyme tests on Julia LTS

This avoids Enzyme compatibility issues with pre-release Julia versions.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Qualify BackTracking with LineSearch. prefix to avoid ambiguity
between LineSearches.BackTracking and LineSearch.BackTracking.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor Author

CI Status Update

All relevant tests are now passing:

  • Enzyme Tests (lts) - PASS - The fix for the BackTracking name conflict worked
  • Tests (1) - PASS
  • Tests (lts) - PASS
  • runic - PASS
  • build - PASS
  • Spell Check - PASS
  • test (alldeps, 1.10) - PASS

The only failure is Tests (pre) which is a pre-existing Julia 1.13-alpha compatibility issue with ReTestItems:

MethodError: no method matching setindex!(::Base.ScopedValues.ScopedValue{Bool}, ::Bool)

This failure also occurs on the main branch (see workflow runs from Jan 4-8, 2026) and is not related to these changes.

ReTestItems has a compatibility issue with Julia 1.13-alpha:
MethodError: no method matching setindex!(::Base.ScopedValues.ScopedValue{Bool}, ::Bool)

See: SciML#47

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas merged commit 9ff7c96 into SciML:main Jan 13, 2026
7 checks passed
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.

3 participants