[Fetch] do not speak warning diagnostics to avoid noisy fetch #1128
Conversation
|
ちなみに，Warnの内容はどういうことをしゃべっているの？ |
|
一番問題だと思うのは、CPUがhigh loadだとしゃべり続けることです。 以下のcpu_monitor.pyで、直近1分 or 5分のCPU loadが一定値を超えるとwarningを出すようになっているのですが、負荷の高い処理を行っているとずっとfetchがしゃべり続けてしまいます。 最初のコメントではrosserialを使っている時に問題が起きると書きましたが、rosserialで出ているメッセージはerrorだったので関係無かったです。 |
|
thresholdを甘くするのはどうかな？
--
◉ Kei Okada
2019年9月12日(木) 15:14 Naoya Yamaguchi <[email protected]>:
… 一番問題だと思うのは、CPUがhigh loadだとしゃべり続けることです。
以下のcpu_monitor.pyで、直近1分 or 5分のCPU
loadが一定値を超えるとwarningを出すようになっているのですが、負荷の高い処理を行っているとずっとfetchがしゃべり続けてしまいます。
またノードをkillしても、過去1分 or 5分のCPU loadはすぐには下がらず、当分しゃべり続けます。
https://github.com/PR2/pr2_robot/blob/kinetic-devel/pr2_computer_monitor/scripts/cpu_monitor.py#L307-L308
最初のコメントではrosserialを使っている時に問題が起きると書きましたが、rosserialで出ているメッセージはerrorだったので関係無かったです。
（たまにerrorが出るけど、rosserialからtopicは出続けていたようです。）
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1128?email_source=notifications&email_token=AADYNXDFCN7XFLOOMDBNR7LQJHM2NA5CNFSM4IVUVRO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6QY4LA#issuecomment-530681388>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADYNXHOVIVPV2BSMP6EFZTQJHM2NANCNFSM4IVUVROQ>
.
|
|
喋らせないようにするためにthresholdを甘くするのは本末転倒感があるので、僕としてはwarningを喋らせないようにするのが良いと思っています。 元々はfetchの腕が落ちる問題の原因を探るためにCPU loadをdiagnosticsに出すようにしていて、thresholdを甘くすると腕が落ちる問題のログ取りにも影響が出るのかなと思いました。 |
|
thresholdを甘くしたら，その甘い所をちょうどうまくつかれる，という，マーフィーの法則ですね．
一方，warning はwarningなので，ユーザに伝わるようにするのが良いともいます．ヒヤリハットの法則．
となると，warningは同じものだったら１０分間喋らない，みたいなプログラムになっちゃう気がするけど．．．
--
◉ Kei Okada
2019年9月12日(木) 17:37 Naoya Yamaguchi <[email protected]>:
… 喋らせないようにするためにthresholdを甘くするのは本末転倒感があるので、僕としてはwarningを喋らせないようにするのが良いと思っています。
元々はfetchの腕が落ちる問題の原因を探るためにCPU
loadをdiagnosticsに出すようにしていて、thresholdを甘くすると腕が落ちる問題のログ取りにも影響が出るのかなと思いました。
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1128?email_source=notifications&email_token=AADYNXE6XLISRJPLYCONYGDQJH5UVA5CNFSM4IVUVRO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6RDXDA#issuecomment-530725772>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADYNXHPHENS7PKSJUONCCLQJH5UVANCNFSM4IVUVROQ>
.
|
ROSのLOGGINGにもthrottleがあるのと同じ作戦ですね cpu_warning.pyの中で、同じwarningは10分ごとにしか話さないようにしようと思います。 |
|
warning.py をいじったほうが，適用範囲が広いのでは？
--
◉ Kei Okada
2019年9月12日(木) 19:02 Naoya Yamaguchi <[email protected]>:
… となると，warningは同じものだったら１０分間喋らない，みたいなプログラムになっちゃう気がするけど．．．
ROSのLOGGINGにもthrottleがあるのと同じ作戦ですね
http://wiki.ros.org/roscpp/Overview/Logging
cpu_warning.pyの中で、同じwarningは10分ごとにしか話さないようにしようと思います。
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1128?email_source=notifications&email_token=AADYNXHB4LNUAJ2F45FSPATQJIHRXA5CNFSM4IVUVRO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6RLGRQ#issuecomment-530756422>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADYNXB54IOFU2WV7FTKNCTQJIHRXANCNFSM4IVUVROQ>
.
|
708yamaguchi
force-pushed
the
ignore-warning
branch
from
d94d16c to
6deac20
Compare
|
warning.pyを編集して、10分間は同じwarningを喋らないようにしました。 あと、僕の勘違いがあって、今のfetchでは既にwarningを喋らない設定になっていました。（最近fetchを触っておらず、すっかり忘れていました） なので、上記のspeak_warn=falseはやめて、10分ごとに同じwarningを喋らせるようにしました。 |
|
@708yamaguchi please resolve conflicts |
708yamaguchi
force-pushed
the
ignore-warning
branch
from
6deac20 to
0baa9cf
Compare
|
I updated. |
|
it seems working correctly on fetch15 |
In this Pull Request, I set
~speak_warnparam asFalseby default, so that our fetch do not speak too much.There are many
ROS_WARNin fetch PC and fetch sometimes speak noisy (e.g. when I use rosserial).This causes fetch user to kill
sound_play.(fetchのwarningがうるさくて僕がしばしばrosnode killしてしまい、他のfetchユーザに迷惑がかかってしまうので、warningは喋らせないようにしました。)
Cc @knorth55