personalrobotics / ada_ros2

ROS2 Hardware Interface and Description for the ADA Robot
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

Remove Octomap in sim mock #41

Closed amalnanavati closed 8 months ago

amalnanavati commented 8 months ago

Description

In practice, the Octomap doesn't work well in sim mock, because the dummy RealSense node just publishes a static depth image irrespective of where the robot arm is. As a result, we ask users to modify sensors_3d.yaml to disable the Octomap in mock. However, modifying a yaml file depending on whether you're simulating a robot or not is undesirable.

Thus, this PR modifies ada_moveit launch so that if sim:=mock, it does not pass in information from sensors_3d.yaml to the MoveGroup.

Testing

TODO: Update README instructions to remove the part about modifying sensors_3d.yaml.