Commit a99979f
Fix Basis compatibility with ODEProblem (SciML#552)
This commit fixes the issue where using a Basis object obtained from DMD
or SINDy as the ODE function in ODEProblem would throw an error about
the 'observed' field not existing.
Changes:
- Added ModelingToolkit interface methods directly to src/basis/type.jl
- Implemented required AbstractSystem interface methods: equations, unknowns,
parameters, get_observed, get_iv, nameof
- Override getproperty to properly handle :observed field access (needed
because ModelingToolkit's getproperty for AbstractSystem tries to use
getvar which doesn't work for Basis)
- Override show method to fix display error related to namespacing flag
Fixes SciML#552
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent c5e4697 commit a99979f
1 file changed
+46
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
559 | 559 | | |
560 | 560 | | |
561 | 561 | | |
| 562 | + | |
| 563 | + | |
| 564 | + | |
| 565 | + | |
| 566 | + | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
| 581 | + | |
| 582 | + | |
| 583 | + | |
| 584 | + | |
| 585 | + | |
| 586 | + | |
| 587 | + | |
| 588 | + | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
| 592 | + | |
| 593 | + | |
| 594 | + | |
| 595 | + | |
| 596 | + | |
| 597 | + | |
| 598 | + | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
0 commit comments