search

SSL-Roots / consai2r2

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

Dev/referee wrapper #47

Closed spiralray closed 4 years ago

spiralray commented 4 years ago

※ PR #46 に依存しています

spiralray commented 4 years ago

ブランチ間の関係がヤバいことになってきたので、本PRで一括してレビューしていただくことは可能でしょうか？

MiraiHattori commented 4 years ago

https://github.com/SSL-Roots/consai2r2/pull/47/checks?check_run_id=349255233#step:6:351 Wno-zero-length-arrayというのはprotobuf専用のフラグのようなものでしょうか？

また，protobufから自動生成されたc++のソースからunused-parameterのwarningが出るので，Wno-unused-parameterを指定してしまう，というのも手かもしれないです．

spiralray commented 4 years ago

はい、protobuf周りのwarningを表示しないために設定しています。 たしかに、-Wno-unused-parameterも使用したほうが良いですね…！

MiraiHattori commented 4 years ago

こちらですが，consai2r2_referee_wrapper内でprotoファイルから自動生成されたpb2があると思うのですが， ubuntu 18.04においてそのファイルをpythonで読んだタイミングで，no module named googleでエラーを吐いて止まります． 不思議に思って調べてみたところ，rosdepで依存関係として入れうるのはpython-protobufですが，実行時はpython3を用いているため，python3-protobufを入れる必要があるようです．

MiraiHattori commented 4 years ago

つまるところ環境構築時にsudo apt-get install python3-protobufを実行するといいのですが，rosdepからそれを解決する方法は思いついていません．ひとまず環境構築ページにsudo apt-get install python3-protobufを入れるとよいかもしれません．

MiraiHattori commented 4 years ago

http://docs.ros.org/independent/api/rosdep/html/contributing_rules.html rosdepにpython3-protobufを入れる気はあまりないのですが，一応やり方のメモです

spiralray commented 4 years ago

別PRから順にmergeされており、conflict解消のためcloseします。