sbgisen / vesc

VESC Interface for ROS
Apache License 2.0
44 stars 33 forks source link

Add respawn options to <node> #2

Closed h-wata closed 3 years ago

h-wata commented 4 years ago

VESCが落ちてしまったあと、/dev/ttyVESC_xx があるときに再起動できるようにrespawnオプションを追加した

実験

h-wata commented 3 years ago

こちら、一旦VESCもFixしましたし、実運用上でVESC nodeが落ちて動かなくなると困るのでrespawnをつけても良いかと思うのですがいかがでしょうか。 VESCのデバッグをするときはオプションを削除するなど手間があるとは思うのですが。

ただ、下記見るとNodeだけ立ち上げ直すと、スピンしてしまうことがあるんですね。 VESCが落ちる環境を再現したいですね。 ref https://github.com/sbgisen/cube/issues/583

ssr-yuki commented 3 years ago

このパッケージに含まれるlaunchでは、パッケージの単体での動作試験も想定しているため、respawnがデフォルトでtrueであるべきではないと判断しました。本件については、cubeリポジトリ内部での対処が望ましいと思いますので、こちらのリポジトリではcloseしたいと思います。