Skip to content
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

add onOriginChanged logic #1037

Open
wants to merge 2 commits into
base: noetic-devel
Choose a base branch
from

Conversation

dorezyuk
Copy link
Contributor

@dorezyuk dorezyuk commented Oct 11, 2020

Hi guys,

I've added an additional function onOriginChanged.

The main idea behind this is to allow to implement layers/cosmaps which are not always rolling but also not entirely static (something in between). So the user can call LayeredCostmap::updateOrigin, if he wishes to move the costmap, but can treat it as static in the remaining time.

The concept fits also well in the current code base - allowing

  1. unify the logic from ObstacleLayer and VoxelLayer
  2. avoid redundant repainting in the StaticLayer (if the robot and thus the origin has not moved)

costmap_2d/plugins/static_layer.cpp Outdated Show resolved Hide resolved
@dorezyuk
Copy link
Contributor Author

@mikeferguson friendly ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants