Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Arrayでのpublish #68

Open
spiralray opened this issue Jul 12, 2020 · 3 comments
Open

Arrayでのpublish #68

spiralray opened this issue Jul 12, 2020 · 3 comments
Labels
enhancement New feature or request

Comments

@spiralray
Copy link
Contributor

8vs8 時など、ロボットの台数が多いとpublish / subscribe時のCPUへの負担が非常に高いです。
arrayでもpublishするようにして、負担を減らすのが望ましいと考えていますが、いかがでしょうか。

@ShotaAk ShotaAk added the enhancement New feature or request label Jul 12, 2020
@ShotaAk
Copy link
Contributor

ShotaAk commented Jul 12, 2020

やはり負荷が高くなるんですね。。
やるとしたら、フィルタリングやチームサイド反転を担うノードで、robot_info_ours, robot_info_theirsのようなトピックにまとめる、とかですかね?

consai2では、raw_vision_detections トピックをrobot_info_{blue, yellow}_{0,1,2~11}の計24トピックに分解していました。
https://github.com/SSL-Roots/consai2/tree/master/consai2_world_observer

@spiralray
Copy link
Contributor Author

感覚的な話で申し訳ないのですが、Pythonでのsubscribeのコストが思った以上に高い気がします。
きちんとベンチマークできたら良いのですが…

consai2r2自体はC++実装が基本なので問題ないのですが、コレをもとに他のチームがAIを作成した際に「重い!」って感じる可能性がありそうです。

現行の個別publish自体は分散処理の観点からも残すべきだと思いますので、+αでarrayでのpublishを足す、というのが良いと思います。(robots_info_blue等)

@ShotaAk
Copy link
Contributor

ShotaAk commented Jul 23, 2020

自分も比較したことはないですが、重そうな感じはしますね・・・

現行の個別publish自体は分散処理の観点からも残すべきだと思いますので、+αでarrayでのpublishを足す、というのが良いと思います。(robots_info_blue等)

#55 のVision_WrapperはCON-SAI2を移植してるのでPython & マルチトピック(not Arrayの意)となってますが、まずは残す方向で見てみます。

その次のステップでC++ & Array トピックのVision_Wrapperを検討します。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants