ros-controls / ros2_controllers

Generic robotic controllers to accompany ros2_control
https://control.ros.org
Apache License 2.0
324 stars 293 forks source link

rqt_joint_trajectory_controller: Use urdf_parser #891

Open christophfroehlich opened 7 months ago

christophfroehlich commented 7 months ago

Maybe we can use the upstream urdf_parser_py? The mentioned issue got fixed years ago..

https://github.com/ros-controls/ros2_controllers/blob/925a690fcb469cb705ec67185cd67c6f1481dd61/rqt_joint_trajectory_controller/rqt_joint_trajectory_controller/joint_limits_urdf.py#L17-L20

bailaC commented 5 months ago

Hello @christophfroehlich , are we going to do this work? If yes, I can give it a try.

christophfroehlich commented 5 months ago

no one is working on that afaik. Thanks for picking this. I just had the idea when we were fixing a bug. Please check if it really makes sense/supports all we need before you start coding!