-
Notifications
You must be signed in to change notification settings - Fork 75
fix(autoware_ekf_localizer): publish diagnostics when ekf_dt is too slow #397
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
base: main
Are you sure you want to change the base?
fix(autoware_ekf_localizer): publish diagnostics when ekf_dt is too slow #397
Conversation
…ng_message and throttle Signed-off-by: RyuYamamoto <ryu.yamamoto@tier4.jp>
Signed-off-by: RyuYamamoto <ryu.yamamoto@tier4.jp>
Signed-off-by: RyuYamamoto <ryu.yamamoto@tier4.jp>
Signed-off-by: RyuYamamoto <ryu.yamamoto@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
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.
You might think like a minor effort to reduce dynamic memory usage, but let’s make sure to declare and use constexpr
strings (char *
or string_view
).
This is second time to mention that. #374 (review)
@RyuYamamoto Could you respond this? |
@youtalk I apologize for the late reply. I'd like to confirm one point regarding your comment. https://github.com/RyuYamamoto/autoware_core/blob/feat/add_diagnostics/localization/autoware_ekf_localizer/src/diagnostics.cpp#L74 |
@RyuYamamoto Please update the readme the-conditions-that-result-in-a-warn-state |
Description
This PR will be merged after the following PR has been merged.
#374
Related links
#374
Parent Issue:
How was this PR tested?
Running Autoware and confirmed that ekf_period has been added to the diagnostics of ekf_localizer.
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.