Yingbo Ma

Results 223 comments of 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 can we make `build_function` check the dimension of the inputs even with `checkbounds=false`?

I thought we already have something similar for the discrete callback thing. @baggepinnen

`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,...