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

ClearCostmapAroundRobot not in Groot #3105

Closed
MrOCW opened this issue Aug 9, 2022 · 3 comments
Closed

ClearCostmapAroundRobot not in Groot #3105

MrOCW opened this issue Aug 9, 2022 · 3 comments
Labels
good first issue Good for newcomers

Comments

@MrOCW
Copy link

MrOCW commented Aug 9, 2022

Bug report

Required Info:

  • Operating System:
    • Jetpack 4.6.1 (Ubuntu 18.04) running Ubuntu 22.04 docker container
  • ROS2 Version:
    • Humble binaries
  • Version or commit hash:
    • latest apt
  • DDS implementation:
    • CycloneDDS

Steps to reproduce issue

Import node palette into Groot from /opt/ros/humble/nav2_behaviour_tree/nav2_tree_nodes.xml

Expected behavior

ClearCostmapAroundRobot to appear in the palette

Actual behavior

Only ClearEntireCostmap is available

Additional information

During contextual recovery, clear entire costmap is used but it takes a very long time for the global costmap to appear again after being cleared, likely due to a large static map, which almost caused a collision as there is no global costmap for the global planner to identify obstacles. Will using ClearCostmapAroundRobot speed up the time needed for the global costmap to appear again?

Replacing ClearEntireCostmap with ClearCostmapAroundRobot manually in the xml file does not produce errors but the costmap doesnt clear around the robot anyways. Manually calling the service doesnt clear it as well. I get this message whenever contextual recovery or recovery subtree gets ticked.
[bt_navigator-11] [WARN] [1660115595.034508823] [bt_navigatornavigate_to_pose_rclcpp_node]: Node timed out while executing service call to global_costmap/clear_around_global_costmap

@SteveMacenski
Copy link
Member

SteveMacenski commented Aug 18, 2022

It can be added to https://github.com/ros-planning/navigation2/blob/main/nav2_behavior_tree/nav2_tree_nodes.xml, would you mind submitting a PR to add this missing BT node so you can use it in groot? I'd be happy to merge it.

ClearCostmapExceptRegion is also missing, might be good to add them both. They both only have 1 input port reset_distance.

@enesbedir1
Copy link
Contributor

@SteveMacenski I created a pr #3155 for the issue
I would be appreciated if you consider reviewing it.

@SteveMacenski
Copy link
Member

Merged!

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

No branches or pull requests

3 participants