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

[Depends on #1378] [jsk_2016_01_baxter_apc] don't use gripper joint to solve IK #1362

Merged
merged 5 commits into from
Apr 30, 2016

Conversation

pazeshun
Copy link
Collaborator

@pazeshun pazeshun commented Apr 28, 2016

screenshot from 2016-04-28 18 57 10
このような姿勢がIKで出てくるのを防止するためのPRです。
具体的には、グリッパーのサーボのIKでの重みを0にし、IKでは動かないようにします。
グリッパーのサーボを動かす際には、(send *baxter* :rotate-gripper :rarm 90 :relative nil)のようにして直接回転させます。

@pazeshun pazeshun added this to the 0.2.6: Stable Grasping Objects with New Gripper milestone Apr 28, 2016
@pazeshun
Copy link
Collaborator Author

IKを解く際の中継点を修正しなければならなかったので、よい中継点を設定することで、手首を回さないとIKが解けない問題( #1341 )を手首を回さなくても解決できるようになりました。
よって、手首を回す行を削除してあります。

fold-pose-lower: [0, -7.27295, 52.6465, -59.9414, 91.582, 123.574, 13.3374, 159.675, 7.27295, 52.6465, 59.9414, 91.582, -123.574, 13.3374, -159.675, 0]
fold-pose-upper: [0, -7.80029, -1.12061, -82.9248, 141.438, 116.477, -6.48193, 8.10791, 35.6955, -0.550063, 174.754, 89.2461, -169.141, 85.136, 4.69734, 90] # this pose of rarm is symmetrical to larm on entrance of bin a
fold-pose-middle: [0, 48.8672, -10.437, -80.6616, 148.645, 80.9033, 1.38428, 18.8745, -48.8672, -10.437, 80.6616, 148.645, -80.9033, 1.38428, -18.8745, 90]
fold-pose-lower: [0, -7.27295, 52.6465, -59.9414, 91.582, 123.574, 13.3374, 159.675, 7.27295, 52.6465, 59.9414, 91.582, -123.574, 13.3374, -159.675, 90]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これを変えても、move-arm-body->binメソッドなどは問題ない?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この中間姿勢は、ik->bin-entranceメソッドでしか使用してなかったはずです。

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

特に、fold-pose-upperについては、変えないとIK普通に解いたときに手首の関節が可動域ギリギリの姿勢が出てきてしまい、そのあとのIKが失敗します。

Copy link
Member

@wkentaro wkentaro Apr 29, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • :ik->bin-entranceでIKの初期姿勢になっている
  • :ik->bin-entranceは他のメソッドで使われている

というのが問題かと思います.

Copy link
Collaborator Author

@pazeshun pazeshun Apr 29, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

今問題になっているのは、ik->bin-entranceで出てくる姿勢がおかしいということです。
それを修正するために、中間姿勢を修正しました。
この中間姿勢はあくまで*baxter*に送られるものであり、これで実機が動くわけではないと考えています。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このPRの副作用を調べるためなので、このPRに必要ですね。

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

わかりました。
といっても、なんのテストを作ればいいのかよくわかっていないのですが、ik->bin-entranceが成功するかどうかをテストすればいいのでしょうか。それとも、よりシーケンシャルな動き(例えば、:move-arm-body->binメソッドをしてから、ik->bin-entranceを使用している各オフセットで解き、それから:view-hand-pose のあと:move-arm-body->order-bin)をテストすれば良いのでしょうか。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変更したfold-poseを使っているのは:ik->bin-entranceだけだったということなので、:ik->bin-entranceだけでいいんじゃないでしょうか。(引数をいくつか変えて)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

わかりました。
#1367 のようなものをik->bin-entranceに対して作ります。できたらmergeしてしまおうと思っています。
大丈夫でしょうか?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

できたら@ban-masaに一度見てもらって下さい.

@pazeshun
Copy link
Collaborator Author

@ban-masa
@wkentaro さんとの話し合いが終わるまで、mergeは待ってください。

@pazeshun
Copy link
Collaborator Author

#1378 でテストを作る用のPRを作り、このPRはそれに依存するようにしました。

@pazeshun
Copy link
Collaborator Author

@ban-masa
reviwお願いします。

@pazeshun pazeshun changed the title [jsk_2016_01_baxter_apc] don't use gripper joint to solve IK [Depends on #1378] [jsk_2016_01_baxter_apc] don't use gripper joint to solve IK Apr 30, 2016
@ban-masa
Copy link
Collaborator

@pazeshun テストの部分ですが、これは_ri_にangle-vectorを送らないので実機は動かないんですよね?このPRはIKが解けるだけでなく、無理な姿勢になっていないかも調べるというものであったと思うのですが、姿勢の良し悪しはテストを実行してIRTViewerを見て判断する感じになるのでしょうか?
コード自体は問題ないように思います。

@pazeshun
Copy link
Collaborator Author

このテストは、特にtravis上で活躍するもので、そもそもIKが解けない状態になっていないことを確認するものです。なので、IRTViewerをみて判断するということはありません。

@ban-masa
Copy link
Collaborator

なるほど。理解してませんでした。問題ないと思うのでmergeします

@ban-masa ban-masa merged commit adb01b2 into start-jsk:gripper-v2 Apr 30, 2016
@pazeshun pazeshun deleted the ik-not-use-gripper branch May 1, 2016 01:44
ban-masa added a commit that referenced this pull request May 2, 2016
[Depends on #1362] [jsk_2016_01_baxter_apc] fix pass to and from bin e
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.

3 participants