-
Notifications
You must be signed in to change notification settings - Fork 125
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
Fix non linear constant turn transition model #844
Conversation
if turn_rate[0] == 0: | ||
turn_rate = Matrix([np.finfo(float).eps]) |
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.
I think, like before, this should check the entire array and set any zero elements to eps.
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.
OK, I've reverted to the original code, but now with a check to ensure the Matrix is float
before trying to fix zero elements.
4ad8a4b
to
c01c14e
Compare
c01c14e
to
92f7f6b
Compare
I found a few bugs in the
ConstantTurn
transition model, which this PR aims to fix.Q_t
did not match the implementation.Q_t
array was erroneously divided by the change in time, which would result in the noise of the turn rate being smaller for a larger change in time. I'm not really sure personally why this value is not instead multiplied by dt, but this seems to be a misunderstanding on my part and not a mistake.I have also raised up a misbehaving comma in the first tutorial as it looked slightly messy.