Universal_Robots_ROS_Driver icon indicating copy to clipboard operation
Universal_Robots_ROS_Driver copied to clipboard

Improve error handling on setIO service

Open fmauch opened this issue 4 years ago • 0 comments

We parse the IO service contents here: https://github.com/UniversalRobots/Universal_Robots_ROS_Driver/blob/930a6cb74d8d420410e24c49364bcc2af50bb5ec/ur_robot_driver/src/hardware_interface.cpp#L852-L880

The current implementation allows setting IO pins with higher numbers than existing, which will silently do nothing. It would be nice to check for the pin limits and make the service fail on an illegal request.

fmauch avatar Jun 24 '21 07:06 fmauch