Skip to content

Raise ModelingToolkit 11.x test floor to 11.21 - #117

Closed
ChrisRackauckas-Claude wants to merge 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:bump-mtk-floor-11.21
Closed

ChrisRackauckas-Claude wants to merge 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:bump-mtk-floor-11.21

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Please ignore this PR until it has been reviewed by @ChrisRackauckas.

What changed

Pure [compat] lower-bound bump:

  • ModelingToolkit = "9, 10, 11"ModelingToolkit = "9, 10, 11.21"

No library code or package version changed.

Why

A declared ModelingToolkit 11.x floor of 11 pins 11.0.0 under julia-downgrade-compat. On Julia 1.10 that release fails to precompile:

ERROR: LoadError: UndefVarError: `ispublic` not defined
  @ ModelingToolkit src/ModelingToolkit.jl:115

Isolated Julia 1.10.11: Pkg.add(name="ModelingToolkit", version="11.0.0") resolves, then precompile dies (tree-sha zsgrz). 11.1.0 still hits ispublic. 11.21.0 loads (same floor as CellMLToolkit / Corleone / CatalystNetworkAnalysis).

Verification

ModelingToolkit Result
11.0.0 (old 11.x floor) precompile UndefVarError: ispublic
11.21.0 (new floor) using ModelingToolkit OK

Not verified: a live julia-downgrade-compat run on this checkout after the bump.

ModelingToolkit 11.0.0 fails to precompile on Julia 1.10 with
UndefVarError: ispublic not defined. 11.21 is the first 11.x series
floor used elsewhere that loads.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member Author

11.21 is still unsatisfiable against this package's DiffEqBase = "7" (registry requires ModelingToolkit 11.22+). Opened #118 with the 11.22 floor and fail-before / pass-after on Julia 1.10.11.

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member Author

Superseded by #118 (DiffEqBase = "7" makes ModelingToolkit 11.21 unsatisfiable; 11.22 is the first that resolves). Closing this one.

Please ignore this until reviewed by @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