Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

制御周期に関するパラメータを統一する #1109

Closed
Naoki-Hiraoka opened this issue Jun 20, 2021 · 1 comment
Closed

制御周期に関するパラメータを統一する #1109

Naoki-Hiraoka opened this issue Jun 20, 2021 · 1 comment

Comments

@Naoki-Hiraoka
Copy link
Contributor

Naoki-Hiraoka commented Jun 20, 2021

現在、制御周期に関するパラメータは、4箇所に2種類の変数名で記述されています.

  • RobotHardware.confファイルのexec_cxt.periodic.rate
  • .confファイルのdt
  • launchファイルのhrpsys_periodic_rate(内部でこの値をもとにexec_cxt.periodic.rateをセット) (ROSが使える環境の場合)
  • rtcd.confファイルのexec_cxt.periodic.rate (ROSが使えない環境の場合)

(choreonoidやgazeboやhrpsys-simulator等のシミュレーターの設定や実機との通信といった、iobより先のコードまで考えるとさらに増えますが、ここでは考えないことにします。)

バラバラに管理されている結果、#408start-jsk/rtmros_tutorials#542 (comment)#1107 (comment)start-jsk/rtmros_tutorials#588 (comment) などで混乱が発生しています。そのため、start-jsk/rtmros_tutorials#542 で提案されているように、制御周期に関するパラメータを1箇所に1種類の変数名で表現するように統一していこうと思います。

具体的には、以下のように統一することを考えています.

問題がありましたら指摘いただけますと幸いです。

@Naoki-Hiraoka
Copy link
Contributor Author

https://github.com/jsk-ros-pkg/trans_system/pull/780#issuecomment-1256850356

なお、
PeriodicExecutionContextやその子クラスを使っている場合、dtはonExecute(RTC::UniqueId ec_id)中で

double dt = 1.0 / this->get_context(ec_id)->get_rate();

とすることで取得可能です.

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

No branches or pull requests

1 participant