Add DelayDiffEq as a sublibrary#3140
Merged
ChrisRackauckas merged 4 commits intoSciML:masterfrom Mar 17, 2026
Merged
Conversation
ChrisRackauckas
changed the title
ChrisRackauckas-Claude
force-pushed
the
add-dde-sublibrary
branch
from
8ee68f0 to
c04586a
Compare
Integrate DelayDiffEq.jl as a sublibrary within the OrdinaryDiffEq.jl monorepo at lib/DelayDiffEq/. The sublibrary is independent — OrdinaryDiffEq does not depend on it. CI is handled automatically by SublibraryCI.yml which detects affected sublibraries from changed files. - Create lib/DelayDiffEq with source from DelayDiffEq.jl - Adapt test/runtests.jl to the new ODEDIFFEQ_TEST_GROUP pattern (Core/QA) - Include all original tests plus multi-algorithm integration test - Preserve original DelayDiffEq LICENSE Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ChrisRackauckas-Claude
force-pushed
the
add-dde-sublibrary
branch
from
c04586a to
e0fb409
Compare
The for-loop assignments to base_group/test_group inside @time begin were silently creating new locals on Julia 1.10 (LTS) due to soft scope rules, causing sublibrary tests to never actually run. Extract the detection logic into a function to avoid scope ambiguity. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The original DelayDiffEq.jl re-exported SciMLBase (via OrdinaryDiffEq), making DDEProblem, DDEFunction, solve, init, ReturnCode etc. available through `using DelayDiffEq`. Tests rely on this. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
2 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
OrdinaryDiffEqDDEsublibrary within the OrdinaryDiffEq.jl monorepoMethodOfStepsandDiscontinuityare now available directly throughusing OrdinaryDiffEqChanges
lib/OrdinaryDiffEqDDE/with all source files from DelayDiffEq.jl, adapted module name, and removed@reexport using OrdinaryDiffEq(since it's now part of OrdinaryDiffEq itself)OrdinaryDiffEqDDEtosrc/OrdinaryDiffEq.jlexports,Project.tomldeps/sources/compatOrdinaryDiffEqDDEandOrdinaryDiffEqDDE_QAtest groups@reexport using OrdinaryDiffEqpattern)test/downstream/delaydiffeq.jlto useOrdinaryDiffEqdirectly instead of the externalDelayDiffEqpackageTest plan
OrdinaryDiffEqDDEmodule loads and precompiles successfullyusing OrdinaryDiffEqDDEandusing OrdinaryDiffEqOrdinaryDiffEqDDEgroup)OrdinaryDiffEqDDE_QAgroup)🤖 Generated with Claude Code