Yingbo Ma
Yingbo Ma
I cannot reproduce it. Did you omit the observed function call?
`sol[model.damper.a_rel]` produces the error. I will fix it.
@shashi could you take a look?
@shashi can we make `build_function` check the dimension of the inputs even with `checkbounds=false`?
Feature Request: allow references to integrator `dt` in `ODESystem`, automatically generate callback
I thought we already have something similar for the discrete callback thing. @baggepinnen
Use `@unpack x = fol`.
`input=true` forces a variable to be a state is intended. Otherwise users won't be able to change it in a callback.
Could you add tests in `test/odesystem.jl`? Also, please test the Jacobian generation as well.
https://github.com/SciML/ModelingToolkit.jl/pull/1616 is a good start. We might need interface changes to DAE observed functions from the DAEFunction side, so that observed functions can depend on derivatives as well.
I think `sol[capacitor.v]` is the wrong interface to get the solution in a performant way. It's inherently extremely dynamic. Maybe we should just have something like: ```julia getcv = getfunction(sol,...