-
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MTK StandardLibrary v2.7.0 broke SecondOrder-block initialization and docs #287
Comments
This PR was breaking but was released as non-breaking |
https://docs.sciml.ai/ModelingToolkitStandardLibrary/stable/ docs still point to v.2.6.0. Building docs is broken, here
|
The docs build is fixed. Yes, we reduced the number of hardcoded zero initializations a bunch. This one probably should have gotten a major, but it's far back enough now that reversing all of the other updates since then would be majorly breaking as well. So I believe this is completed, please feel free to continue the conversation if it's not. |
Describe the example
ODE initialization broke for
SecondOrder
block in ModelingToolkitStandardLibrary v2.7.0. Similar usage ofFirstOrder
-block still works.Minimal Reproducible Example 👇
Error & Stacktrace⚠️
Not Working Environment (please complete the following information):
using Pkg; Pkg.status()
using Pkg; Pkg.status(; mode = PKGMODE_MANIFEST)
versioninfo()
Working Environment (please complete the following information):
using Pkg; Pkg.status()
using Pkg; Pkg.status(; mode = PKGMODE_MANIFEST)
versioninfo()
Additional context
SecondOrder
is initialized withguess
https://github.com/SciML/ModelingToolkitStandardLibrary.jl/blob/v2.7.0/src/Blocks/continuous.jl#L170-L173whereas
FirstOrder
is not https://github.com/SciML/ModelingToolkitStandardLibrary.jl/blob/v2.7.0/src/Blocks/continuous.jl#L125-L126Seems to work with either
or
The text was updated successfully, but these errors were encountered: