Skip to content

Show warning if control loop takes longer duration than expected. - #1

Open
mitsuharu-kojima wants to merge 1 commit into
yoshito-n-students:masterfrom
mitsuharu-kojima:show_warning_longer_control_loop
Open

Show warning if control loop takes longer duration than expected.#1
mitsuharu-kojima wants to merge 1 commit into
yoshito-n-students:masterfrom
mitsuharu-kojima:show_warning_longer_control_loop

Conversation

@mitsuharu-kojima

Copy link
Copy Markdown
Contributor

制御が破綻していることに気づきやすいように警告をだすようにしました。

@yoshito-okada

Copy link
Copy Markdown
Contributor

PRありがとうございます.

内容確認しましたが,以下の修正が必要と思います.ご検討いただけないでしょうか?

  • 「実際の制御周期が設定より遅れている」というのはuse_expected_periodの値と独立に起きる可能性がある問題であるので,use_expected_period:=trueの時以外でも警告を出すべきである
  • 警告の条件がexpected_period < actual_periodは厳しすぎるのではないか(実際の実行周期は多少揺らぐため).例えば,実際が設定の2倍を超えたときに「2倍以上遅れている」と警告を出すのはどうか.

@yoshito-okada yoshito-okada added the enhancement New feature or request label Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants