Skip to content

Select the DiffEqFlux downstream test suite#1271

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:agent/fix-diffeqflux-downstream-group
Jul 16, 2026
Merged

Select the DiffEqFlux downstream test suite#1271
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:agent/fix-diffeqflux-downstream-group

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Ignore this PR until it has been reviewed by @ChrisRackauckas.

Summary

  • replace the stale DiffEqFlux GROUP=DiffEqFlux selection with the target's curated All suite
  • leave the other downstream matrix entries unchanged

DiffEqFlux declares BasicNeuralDE, AdvancedNeuralDE, Newton, and Layers, with All selecting those four groups. It does not declare a DiffEqFlux group, so the old job did not select a real target test group.

Local verification

  • actionlint .github/workflows/Downstream.yml
  • julia +1.12 --startup-file=no -m Runic --check .
  • git diff --check
  • exact Julia 1.12 downstream replay with GROUP=All, coverage enabled, and Optimization resolved from ../repos/Optimization.jl

The exact replay completed with Testing DiffEqFlux tests passed. Its final and longest testset reported:

Test Summary:   | Pass  Total      Time
Stiff Nested AD |    3      3  73m01.4s

The full downstream replay took approximately 3h49m.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review July 16, 2026 09:41
@ChrisRackauckas
ChrisRackauckas merged commit 876ddf6 into SciML:master Jul 16, 2026
47 of 48 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.

2 participants