-
Notifications
You must be signed in to change notification settings - Fork 205
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
std::copy fix - OccupancyGridUpdate - Data is not being processed correctly #895
Conversation
Issue: #896 |
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.
@Eric-Bwr Thanks for the patch. I've left a requested change inline. Can you test that out and verify that it fixes the original issue for you?
rviz_default_plugins/src/rviz_default_plugins/displays/map/map_display.cpp
Outdated
Show resolved
Hide resolved
…_display.cpp Co-authored-by: Chris Lalancette <clalancette@gmail.com>
The warning on Windows is unrelated to this PR (and has already been fixed elsewhere). So going ahead and merging this; thanks @Eric-Bwr ! |
No problem :) Thank you aswell! |
…rectly (ros2#895) * std_copy_fix Co-authored-by: Eric Böwer <e.boewer@kalverkamp.de> Co-authored-by: Chris Lalancette <clalancette@gmail.com>
This fixes the updating of the costmap. As described in the Issue.