search

nobleo / full_coverage_path_planner

Full coverage path planning provides a move_base_flex plugin that can plan a path that will fully cover a given area
Apache License 2.0
547 stars 156 forks source link

Galactic code review #29

Closed cesar-lopez-mar closed 1 year ago

cesar-lopez-mar commented 3 years ago

Made most changes suggested in PR25 https://github.com/nobleo/full_coverage_path_planner/pull/25

cesar-lopez-mar commented 3 years ago

ping

Timple commented 3 years ago

These improvements all look good, so approving.

But this PR is of course only a review of the improvements, not a full review. So that issue stays open.

I'll pick that up later this week.