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

[py] Add a warning inline doc for setJointAngles. #1148

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

130s
Copy link
Contributor

@130s 130s commented May 31, 2017

Inline で書いている通り,tork-a/rtmros_nextage#332 (comment) の指摘を python の inline doc に追加しています.プログラマティックな作用は一切無いです.

指摘中にある"やってはいけない"を抑止する効果を追加しようとすると Python よりも上流で何かする必要があるんだと思いますが,ちょっと私にはわかりません.

As added in inline doc, this adds a doc for the "do-not-do" pointed out at tork-a/rtmros_nextage#332 (comment). No programmatic effect, so changes to somewhere upstream than Python should be made.
@130s
Copy link
Contributor Author

130s commented Jun 3, 2017

Travis 通ったので review お願いします

@k-okada
Copy link
Contributor

k-okada commented Jun 5, 2017

基本的な話はhttps://github.com/fkanehiro/hrpsys-base/issues/36で議論したように、グループを指定(add group)すると元のもの(setJointAngles)は上書きされる.
わかりづらいのは、setTargetPose はグループを前提にしている(処理を考えるとそうなですが)ということでしょうか.
なので、この話を、どこに書いたらいいのかわからないですが.たとえばhironxに関しては必ずグループが指定されるので、#36によれば、remove groupしなければ、そもそもsettargetposeを利用する場面はおとずれないはず、なので、そちら側でwarningだすというのもあるかもしれないです.

@130s
Copy link
Contributor Author

130s commented Jun 6, 2017

#36によれば、remove groupしなければ、そもそもsettargetposeを利用する場面はおとずれないはず

これはどうしてでしょうか?すみませんが #36 何度読んでも解らないです.

setTargetPose の第一の利用目的は姿勢の指定だと思っています.API のユーザにとって,グループが remove されたかどうかという内部情報は気にしないと思います.
少なくとも現状の API doc にはグループ削除云々が書いてない気がするのでこの PR を出しています.

そちら側でwarningだすというのもあるかもしれないです.

どこでどういう内容を出すべきかもよく解らないです.
hrpsys_config.py だけ見ると,group を削除するみたいなのは見当たらない感じがしていますが.

いま時点でこの PR は setJointAngles の api doc に警告的な文を足していますが,それはあるコマンド順番の後に setJointAngles を実行すると実現できてしまう問題への暫定対処のつもりです.なので,#1148 (comment) にも書いた通り現状の PR がベストとは思いませんが,setJointAngles に何か対応するのは必須に思えます.

@k-okada
Copy link
Contributor

k-okada commented Jun 7, 2017

#36によれば、remove groupしなければ、そもそもsettargetposeを利用する場面はおとずれないはず

すいません.そもそもsetjointanglesを利用する場面は訪れないはず、のまちがいでした.

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

Successfully merging this pull request may close these issues.

2 participants