-
Notifications
You must be signed in to change notification settings - Fork 284
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
Fixed period computation in test #693
Fixed period computation in test #693
Conversation
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.
well-spotted!
Please run |
Sorry, I know this, did it several times, and for the PR (which is only a single change compared to many I did), I did a cherry-pick and apparently this introduced a line. I forgot to re-run. I think I had a reason to not "install" the pre-commit but I forgot which one. |
I find precommit on every commit to be slowing my work down and be generally annoying so I also never actually run |
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.
thanks!
@Mergifyio backport humble |
✅ Backports have been created
|
(cherry picked from commit de7083e)
(cherry picked from commit 88c3463)
(cherry picked from commit 88c3463)
As discussed and confirmed in #686 there seem to be a wrong computation of the period (dt) passed to
traj_controller->update
in theUpdateController
test helper function.Fixing it does not make tests fail, but is, first necessary to avoid keeping wrong computations, and second for future up-coming improvement that relies on dt to update the state interface as well.