search

jsk-ros-pkg / jsk_roseus

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

enable to publish info, currently we publish node's function document, or node name #716

Open k-okada opened 2 years ago

tkmtnt7000 commented 2 years ago

キッチンデモにこのPRをいれて試してみました．2022/08/23 22:51 の Message from Fetch1075がタイトルのメールです． docstringをつけたものは小文字，大文字になっているのは関数から引っ張ってきた名前です． これまでにsmachで書かれたデモに最小限の変更で適用するには良さそうな気がします．

https://user-images.githubusercontent.com/67531577/186182436-693236f4-e1c8-45ab-abd3-1657a7058371.mp4

tkmtnt7000 commented 2 years ago

The part of main inprementation is here. https://github.com/tkmtnt7000/jsk_robot/blob/360a8b70b57b680eb78f7b18cf2f27f0fc5d152f/jsk_fetch_robot/jsk_fetch_startup/euslisp/navigation-utils.l#L643-L674 and https://github.com/tkmtnt7000/jsk_robot/blob/360a8b70b57b680eb78f7b18cf2f27f0fc5d152f/jsk_fetch_robot/jsk_fetch_startup/euslisp/navigation-utils.l#L739-L741

k-okada commented 2 years ago

@tkmtnt7000 thank you for checking, I think DESCRIPTION/IMAGE is designed to tell users about how they think and what they do. but INFO is for developpers. so, I think message body is

キッチンを見に行くよ
IMAGE
キッチンの様子を見たよ
IMAGE

---
INFO : <time>
INFO : <time>
.....
tkmtnt7000 commented 2 years ago

Thank you for advice. I change smack_to_mail to display INFO for developers.

k-okada commented 2 years ago

https://github.com/jsk-ros-pkg/jsk_robot/pull/1568#pullrequestreview-1083013469 にコメントしましたが，DESCRIPTION/IMAGE は開発者ではない人，が読むメッセージとして確認するのが良いと思います。 INFOは開発者向けでよいとおもいます。

開発者の定義はLispが読める人，そのプログラムを自分で治す可能性がある人，です。