-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
rm old discrete system #2413
rm old discrete system #2413
Conversation
Rebase |
rm more stuff
isdifference(expr) = isoperator(expr, Difference) | ||
isdifferenceeq(eq) = isdifference(eq.lhs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These functions are used in other places. Is there a corresponding implementation for the new discrete system? hasshift
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isdifference
is no longer the correct question to ask, you can have an equation that is not a difference equation, but it still belongs to the discrete time domain.
You now have has_discrete_domain
and other similar functions in the same file, but they require clock inference to have been run on the model before they are fully usable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's an example of how clock inference is manually called and the inferred time domain of equations and variables tested
https://github.com/SciML/ModelingToolkit.jl/blob/master/test/clock.jl#L180C1-L194C32
I don't have the permissions to push here, so I opened another PR branched off of this one #2440 |
No description provided.