microsoft / spot-ros-wrapper

ROS Wrapper for the Boston Dynamics Spot robot
MIT License
64 stars 13 forks source link

[Design] wrapper bypasses estop system completely. #12

Open mklingen-bd opened 4 years ago

mklingen-bd commented 4 years ago

https://github.com/microsoft/spot-ros-wrapper/blob/7a22c6715de60d41b50d43e8e63a708c7e6b4bab/spot_ros_interface/scripts/spot_ros_interface.py#L493

The estop system was designed so that multiple devices (tablet, laptop, web interface, etc.) could become an authoratative emergency stop source, and stop the robot when triggered or when comms to the robot was lost. The ROS wrapper automatically acquires the estop and keeps it alive, negating this purpose.

This is another design question, because estops are not a ROS concept. The estop service could be made into it's own ROS service, or a 3rdparty estop system could be used in conjunction with ROS.