-
Notifications
You must be signed in to change notification settings - Fork 88
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
Divide output layer class into specific classes #633
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
When Costmap3dLayerStopPropagation
is not used on top of the dynamic layers, updates in the static layers should be propagated to the dynamic layers.
Without Costmap3dLayerStopPropagation
, /costmap_update
contains both static obstacles and dynamic obstacles.
This mode is needed when planner_3d
is used with overwrite_cost: true
and layer's overlay_mode: overwrite
.
(It is for the use cases that robot run over the static map obstacles when local observation can clarify the place is safe to travel)
I didn't change the algorithm of making
|
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.
I got the meaning of the change now. LGTM
costmap_cspace/src/costmap_3d.cpp
Outdated
{ | ||
if (update) | ||
{ | ||
publishDebug(*map); |
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.
I think updated map should be reflected to the debug output instead of the static map
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.
Done. Please take another look.
[190] PASSED on melodicAll tests passed
[190] PASSED on noeticAll tests passed
|
[191] PASSED on noeticAll tests passed
[191] PASSED on melodicAll tests passed
|
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.
LGTM
This PR divides Costmap3dLayerOutput into Costmap3dStaticLayerOutput and Costmap3dUpdateLayerOutput.
This change can omit unnecessary generation of costmap_cspace_msgs::CSpace3DUpdate when static map is updated.