fix(localization_error_monitor): fixed header stamp #7174
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Currently,
localization_error_monitor
outputs a timestamp in the diagnostic message header with the node'sthis->now()
set. However, we are interested in the time when the message monitored by thelocalization_error_monitor
node was published. So I changed the timestamp to be set, fromthis->now()
toinput_msg->header.stamp
.Tests performed
I ran logging_simulator and parsed the
/diagnostics
topic from the saved rosbag.I have confirmed that all timestamps in
localization: localization_error_monitor
have a match inlocalization: ekf_localizer
.Effects on system behavior
There are no effects on system behavior.
Interface changes
There are no interface changes.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.