search

magazino / move_base_flex

Move Base Flex: a backwards-compatible replacement for move_base
https://uos.github.io/mbf_docs/
BSD 3-Clause "New" or "Revised" License
422 stars 154 forks source link

fix wrong (initial) global planner path cost #320

Closed ChristofDubs closed 1 year ago

ChristofDubs commented 1 year ago

There is this piece of code that sets a plan's cost in case the planner_->makePlan doesn't set the cost:

https://github.com/magazino/move_base_flex/blob/9ccecd1426790a8ab66f8394a94f951946f14fc0/mbf_abstract_nav/src/abstract_planner_execution.cpp#L304-L307

This however only works properly if cost is initialized to 0