-
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
costmap_cspace: add linear_spread_min_cost parameter #719
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## master #719 +/- ##
==========================================
- Coverage 88.65% 88.32% -0.34%
==========================================
Files 60 60
Lines 4435 4444 +9
==========================================
- Hits 3932 3925 -7
- Misses 503 519 +16
|
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.
The change looks good to me, but I'm not sure the word cutoff cost
fits well
@f-fl0 any thoughts on the parameter name?
|
How about |
I think this is better, especially with the diagram. |
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.
Could you add the parameter to
https://github.com/at-wat/neonavigation/blob/master/costmap_cspace/README.md#parameters-1
with the diagram?
Sorry I meant the diagram from the PR description. |
I meant to put the diagram in the PR description to README |
[507] PASSED on noeticAll 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
linear_spread_min_cost
is the cost of edge grids spread from an occupied grid. (See Costmap3dLayerOutput.LinearSpreadMinCost)This parameter can make the costs of grids near obstacles higher to avoid other costs such as preferences overriding the costs of these grids.