Update :reset-force-moment-offset to use :off force/moment vector

[snozawa]

Update :reset-force-moment-offset to use :off force/moment vector

[snozawa]

travis passed, but review required??
#977

[k-okada]

yes, ask someone for review.

◉ Kei Okada

2016-10-26 9:15 GMT+09:00 Shunichi Nozawa [email protected]:

travis passed, but review required??
#977 #977

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#983 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeG3AbYzhalme9faB71-wLqwPa1YqYcks5q3pu0gaJpZM4KgHYH
.

[snozawa]

@k-okada,
了解です．
ちなみに，これはいつからのrtmros_commonの方針変更でしょうか？
全員のＰＲがreviewがいるように設定変更がされてるという理解でいいでしょうか？

[snozawa]

@mmurooka
could you please review this?

[mmurooka]

lgtm

[snozawa]

Thank you for reviewing.@mmurooka

[mmurooka]

こちらrtmros_commonをupdateしていない環境で作った*ri*で，
(send *ri* :reset-force-moment-offset-arms)すると，エラーにはならずオフセットが更新されますが，
off_lfsensorのオフセット除去値がゼロにはならない（むしろ大きくなることがある）ので要注意でした．
git pull origin/master したら，ちゃんとオフセット除去してゼロになりました．

@iory , @ban-masa, @oshiroy, @YuOhara , @rkoyama1623

[k-okada]

バージョンをちゃんとつけて、Warning出すようにするといいですね．pythonだとこうやっています． https://github.com/fkanehiro/hrpsys-base/blob/master/python/hrpsys_config.py#L1232

…

-- ◉ Kei Okada 2016年11月28日 22:28 Masaki Murooka <[email protected]>:

こちらrtmros_commonをupdateしていない環境で作った*ri*で， (send *ri* :reset-force-moment-offset-arms)すると，エラーにはならずオフセットが更新されますが， off_lfsensorのオフセット除去値がゼロにはならない（むしろ大きくなることがある）ので要注意でした． git pull origin/master したら，ちゃんとオフセット除去してゼロになりました． @iory <https://github.com/iory> , @ban-masa <https://github.com/ban-masa>, @oshiroy <https://github.com/oshiroy>, @YuOhara <https://github.com/YuOhara> , @rkoyama1623 <https://github.com/rkoyama1623> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#983 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAeG3I3lf6g5OmAP5sw-mbBcrRhfC5-nks5rCtcWgaJpZM4KgHYH> .

[snozawa]

バージョンをちゃんとつけて、Warning出すようにするといいですね．pythonだとこうやっています．

そうですね，次の機会から気を付けます．

こちらrtmros_commonをupdateしていない環境で作ったriで，
(send ri :reset-force-moment-offset-arms)すると，エラーにはならずオフセットが更新されますが，
off_lfsensorのオフセット除去値がゼロにはならない（むしろ大きくなることがある）ので要注意でした．

これってこうなりました？
armsとついてる関数でupdateしたら，fsensorはupdateされないと思ってますが，されてるというバグなんでしょうか．

[mmurooka]

off_lfsensorのオフセット除去値がゼロにはならない（むしろ大きくなることがある）ので要注意でした．

すみません，off_lhsensorの書き間違いでした．
17号機（cとvの中が新しめ）に対して，僕の席のPC（rtmros_commonが古かった）で*ri*を作ったら起きました．

[snozawa]

なるほど，了解です．