Skip to content

Remove stale OptimizationMOI MTK imports#1248

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:codex/fix-optimizationmoi-julia-pre-import
Jul 8, 2026
Merged

Remove stale OptimizationMOI MTK imports#1248
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:codex/fix-optimizationmoi-julia-pre-import

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor

This PR should be ignored until reviewed by @ChrisRackauckas.

Summary

  • remove stale direct imports from ModelingToolkitBase in OptimizationMOI
  • keep using the existing MTK = ModelingToolkitBase alias for public access
  • avoids Julia 1.13 precompile output warning that ModelingToolkitBase.mergedefaults is undeclared at import time

Verification

  • On clean master, timeout 3600 /home/crackauc/.juliaup/bin/julia +1.13 --project=lib/OptimizationMOI -e 'using Pkg; Pkg.instantiate(); Pkg.precompile(); using OptimizationMOI; println("OptimizationMOI loaded")' loaded but emitted WARNING: Imported binding ModelingToolkitBase.mergedefaults was undeclared at import time during import to OptimizationMOI.
  • After this patch, the same Julia 1.13 command passed with OptimizationMOI loaded and no ModelingToolkitBase.mergedefaults undeclared-import warning.
  • timeout 3600 /home/crackauc/.juliaup/bin/julia +1.13 --project=lib/OptimizationMOI -e 'using Pkg; Pkg.test(; julia_args=["--check-bounds=yes", "--compiled-modules=yes", "--depwarn=yes"], force_latest_compatible_version=false)': passed, Core | 40 pass, 2 broken, 42 total.

Avoid importing ModelingToolkitBase bindings that are not declared at import time on newer Julia versions. The module already accesses ModelingToolkitBase through the MTK alias.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review July 8, 2026 13:46
@ChrisRackauckas ChrisRackauckas merged commit e535903 into SciML:master Jul 8, 2026
41 of 49 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