-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add point that reset variables must be initialised? #329
Comments
How do you mean "it must be initialised" ? |
It must have an |
Yeah I think that that's the only way the reset rules make sense. We can't have both "x is always 3" and "set x to 7". Probably wherever we explain the difference between eternal truths and initial values would be the place to bring this up? |
Another confusing thing: It's not that initial values are only true at t=0, it's that they're true until changed (whether by integration, maths, or resets makes no difference). The "initial" throws me a little, as I associate it only with integration. Maybe we can use the attribute |
That's cool! |
Initially, x=0, until changed by a reset or ODE ? |
... or a change in another variable in an equation? Anything that has a value that can change for any reason needs to be initialised directly through either |
how so? |
A = B + C
Can we not just say: x=0 until changed by something? Ditch the initial part? |
Is "initial" not a shorthand for "when the vector t is some unspecified value t0" ? |
It implies that it's a state variable (to me anyway), rather than just a regular variable that you need to give a value to. But I guess this could be cleared up with some clarity in whatever section in chapter 3 of the normative, and some good examples in the informative. |
What do people feel about adding a point to either:
- 2.9 The reset element, or
- 3.11 Interpretation of resets
that says that the reset variable must be initialised? I think it would make a huge amount of the operation of the resets clearer, it's easy to validate, and the resets won't actually work without it ...?
The text was updated successfully, but these errors were encountered: