SSL-Roots / consai2r2

https://github.com/SSL-Roots/consai_ros2 へ開発を移行しています。
MIT License
8 stars 7 forks source link

追加設定なしでgrSimと通信できるようにする #41

Open MiraiHattori opened 4 years ago

MiraiHattori commented 4 years ago

consai2ではgrSimのvisionとconsai2_receiverの通信のため consai2_description/param/game.yamlのvision_portを10006から10020に書き換える or grSimの10020を10006に書き換える必要がありました. これがチュートリアルを兼ねているなら,通信を行うアプリケーションのipとportと通信方式を表した図を追加した方がよいと考えています. そうでないなら,grSimに合わせてデフォルト設定を10020に変えておくのがいいと思うのですが,いかがでしょうか?

MiraiHattori commented 4 years ago

そもそもgrSimのvision multicast portが10020になっている理由が気になってSSLのdiscordで聞いてみたところ, multicast portは(ヒューマンエラーで)混線しやすく危ないから,よく使われるポートと一致すると困る,ということのようです.

MiraiHattori commented 4 years ago

また,国内の大会では10006がよく使われるということなので,

これがチュートリアルを兼ねているなら,通信を行うアプリケーションのipとportと通信方式を表した図を追加した方がよいと考えています.

をするとよいかな,と思っています.

MiraiHattori commented 4 years ago

https://mermaidjs.github.io/mermaid-live-editor/#/edit/eyJjb2RlIjoiZ3JhcGggVERcbiAgICBjb25zYWkycjJfc2VuZGVyIC0tPnxVRFAgMTI3LjAuMC4xOjIwMDExfCBnclNpbS1jb21tYW5kXG7jgIBnclNpbS1yb2JvdCAtLT4gfFVEUCAxMjcuMC4wLjE6MzAwMTF8IG5vbmVcbiAgICBzc2wtZ2FtZS1jb250cm9sbGVyIC0tPiB8VURQIDIyNC41LjIzLjE6MTAwMDN8IGNvbnNhaTJyMl9yZWNlaXZlclxuICAgIGdyU2ltLXZpc2lvbiAtLT4gfFVEUCBtdWx0aWNhc3QgMjI0LjUuMjMuMjoxMDAwNiBvciAxMDAyMHwgY29uc2FpMnIyX3JlY2VpdmVyXG4gICAgZ3JTaW0tdmlzaW9uIC0tPiB8VURQIG11bHRpY2FzdCAyMjQuNS4yMy4yOjEwMDA2IG9yIDEwMDIwfCBzc2wtZ2FtZS1jb250cm9sbGVyIiwibWVybWFpZCI6eyJ0aGVtZSI6ImRlZmF1bHQifX0

これだと伝わらないとは思っているのですが雰囲気だけ

ShotaAk commented 4 years ago

:god:

ShotaAk commented 4 years ago

ありがとうございます!情報収集までしてくださって、、、 あるとないとじゃ理解度が大違いです。