Skip to content

[develop/fetch] [jsk_fetch_startup] [kitchen_demo] always store photo in smach#1750

Merged
mqcmd196 merged 1 commit intojsk-ros-pkg:develop/fetchfrom
mqcmd196:PR/always-take-pic
Dec 5, 2022
Merged

[develop/fetch] [jsk_fetch_startup] [kitchen_demo] always store photo in smach#1750
mqcmd196 merged 1 commit intojsk-ros-pkg:develop/fetchfrom
mqcmd196:PR/always-take-pic

Conversation

@mqcmd196
Copy link
Member

@mqcmd196 mqcmd196 commented Dec 4, 2022

Background

Currently, Fetch records images in SMACH_STATE only when detects some objects. I want to record all images for processing them with Language model

@mqcmd196 mqcmd196 requested a review from tkmtnt7000 December 4, 2022 06:36
@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Dec 4, 2022

I think it's ok to merge this PR soon for the master's thesis experiment, but we should think about notification level #1648

FYI:
#1727

Koki Shinjo, 11月14日, 16:56, 編集済み
chatの話題流れがちなので、ロボットがデモの様子を報告するチャットルームを分けたいです

Naoto Tsukamoto, 11月15日, 7:34
ちょっとうるさくなっている気がするので、チャットルームを分けるか優先度に応じて通知を抑制するかした方が良さそうですね

岡田慧, 11月15日, 7:39
分けるのは新しいデバッグストリームを作るだけなので、変化/発見がある時だけ画像を添付してみましょう

また大日向君/塚本君的にはロボットが喋りかけるスペースは一箇所に集めて、将来的にはロボット間のインタラクションがどう生まれてほしいか観察するのが良いと思うんじゃないかな?

Naoto Tsukamoto, 11月15日, 7:43
ありがとうございます
変化があるときだけ送るような形で考えてみます

岡田慧, 11月15日, 7:44
まずはテキストは今のままで画像だけ添付を切り替えるのは10分で対応できそう

Naoto Tsukamoto, 11月15日, 7:44
はいすぐ対応します

Naoto Tsukamoto, 11月15日, 7:46, 編集済み
>また大日向君/塚本君的にはロボットが喋りかけるスペースは一箇所に集めて、将来的にはロボット間のインタラクションがどう生まれてほしいか観察するのが良いと思うんじゃないかな?

最近使われてないですが
昔jsk robot botみたいなスペースがあったような記憶があります
ここに貯めてみるのもいいかもしれません

@mqcmd196
Copy link
Member Author

mqcmd196 commented Dec 5, 2022

I think it's ok to merge this PR soon for the master's thesis experiment, but we should think about notification level #1648

Yes, for handling it

@mqcmd196 mqcmd196 merged commit 8f99039 into jsk-ros-pkg:develop/fetch Dec 5, 2022
@mqcmd196 mqcmd196 deleted the PR/always-take-pic branch December 5, 2022 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants