Import FunctionOperator from SciMLOperators in Core tests - #1127
Import FunctionOperator from SciMLOperators in Core tests#1127ChrisRackauckas-Claude wants to merge 1 commit into
Conversation
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
|
Investigation/validation scratchpad:
|
Ignore until reviewed by @ChrisRackauckas.
Summary
FunctionOperator,MatrixOperator, andWOperatordirectly from their owning package, SciMLOperators, at the start oftest/Core/default_algs.jl.Why
SciMLBase's re-export cleanup removed the transitive
FunctionOperatorbinding. The boundary is SciMLBase commitb9c3507a2670fae804bf627ca32354efeee8dc15: its parent still exposes the binding, while that commit does not. SciMLOperators v1.25.0 publicly exports and documentsFunctionOperator, so the Core test should import it from SciMLOperators rather than rely on a transitive re-export.Verification
test/Core/default_algs.jlstops at line 167 withUndefVarError: FunctionOperator not defined.Default Alg Tests | 109 pass | 109 total.git diff --check upstream/main...HEADexited 0.I also ran the official
GROUP=Coreharness with the now-merged #1125 and #1126 fixes applied. It passed the default-algorithm tests (109/109) and continued through the ForwardDiff overload tests (128/128), then stopped at an independent clean-main regression intest/Core/traits.jl:UndefVarError: IdentityOperator not defined in SciMLBase. That owner-import regression needs a separate focused follow-up and is intentionally not included here.