Add linear acceleration to synchronizer #205
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.
This is a potential bug since the acceleration publisher was added in #129 and since then
getState
also retrieves the linear acceleration:https://github.com/clearpathrobotics/fuse/blob/def59088a0ee3ff148a4e5a2d3abb8f853bbbb2d/fuse_models/src/odometry_2d_publisher.cpp#L234-L236
However, I forgot to add the linear acceleration to the synchronizer that's used to retrieve the latest common timestamp in https://github.com/clearpathrobotics/fuse/blob/def59088a0ee3ff148a4e5a2d3abb8f853bbbb2d/fuse_models/src/odometry_2d_publisher.cpp#L98
This fixes this potential issue.