jsk-ros-pkg / jsk_roseus

ROS EusLisp Client
http://wiki.ros.org/roseus/Tutorials
17 stars 56 forks source link

[eustf.l] fix comment of :lookup-transform #650

Closed Naoki-Hiraoka closed 3 years ago

Naoki-Hiraoka commented 3 years ago

The original comment was incorrect. :lookup-transform returns the transform from target_frame to source_frame. (not from source_frame to target_frame)

https://github.com/ros/geometry/issues/108 https://answers.ros.org/question/229463/confusing-tf-transforms/ https://uenota.github.io/dronedoc/appendix/trouble_shooting/subpages/ros/reversed_tf.html https://github.com/jsk-ros-pkg/jsk_recognition/issues/1269

Naoki-Hiraoka commented 3 years ago

transform from A to B という英文を、 「A座標系から見た、Bの位置姿勢」と解釈するとこのPRの記述( transform from target_frame to source_frame)が正しく、 「A座標系で表現された座標から、B座標系で表現された座標へと変換するために左から掛けるべき変換行列」と解釈するともとの記述( transform from source_frame to target_frame)が正しくなることに気づきました。

transform from A to B ではない、わかりやすい記述に変えるべきだと思い、 the coordinate of source_frame expressed in target_frame coordinate system へと変更しました。