Universal_Robots_ROS2_Driver icon indicating copy to clipboard operation
Universal_Robots_ROS2_Driver copied to clipboard

Use pose_broadcaster to publish the TCP pose

Open fmauch opened this issue 1 year ago • 4 comments

This uses the PoseBroadcaster currently written by @RobertWilbrandt. As he wants to contribute that to ros2_controllers, this will stay in a draft state until it is merged upstream.

fmauch avatar Sep 19 '24 18:09 fmauch

High-level question: would it perhaps be better to have the hardware interface offer a quaternion instead of an Euler angle triplet?

That would seem cleaner to me, in the sense of it being more of an N-to-1 mathematical abstraction.

While they aren't necessarily very human friendly, quaternions have definite benefits mathematically, and making it the responsibility of the hardware interface/driver to convert whatever internal orientation representation it uses to this 'intermediary representation' would seem to make sense.

gavanderhoorn avatar Sep 26 '24 08:09 gavanderhoorn

High-level question: would it perhaps be better to have the hardware interface offer a quaternion instead of an Euler angle triplet?

That would seem cleaner to me, in the sense of it being more of an N-to-1 mathematical abstraction.

While they aren't necessarily very human friendly, quaternions have definite benefits mathematically, and making it the responsibility of the hardware interface/driver to convert whatever internal orientation representation it uses to this 'intermediary representation' would seem to make sense.

I was discussing that with @RobertWilbrandt, as well and I would agree. But I think that's a question to discuss once the PR on ros2_controllers is there.

fmauch avatar Sep 26 '24 09:09 fmauch

Oh, just noticed #856 actually does it that way.

gavanderhoorn avatar Sep 26 '24 09:09 gavanderhoorn

I don't have any strong opinions on this - Going through REP 103 i went through the options and preferred RPY because the values cannot represent any invalid state (while i would expect that a quaternion should be normalized). From an application point of view i agree, so i changed the representation in pose_broadcaster now.

RobertWilbrandt avatar Sep 30 '24 16:09 RobertWilbrandt

Binary builds fail as the necessary changes in ros2_control have just been merged into master.

fmauch avatar Oct 27 '24 18:10 fmauch