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

STを入れるとおかしな姿勢になる #191

Open
robograffitti opened this issue Jun 22, 2015 · 16 comments
Open

STを入れるとおかしな姿勢になる #191

robograffitti opened this issue Jun 22, 2015 · 16 comments

Comments

@robograffitti
Copy link
Member

gazeboのsample robotでroseusから

(send ri :start-auto-balancer)
(send ri :start-st)
とすると、:start-stをした時に、
画像のように左肩が下がります。

:go-posで歩かせると当然コケますが、どこかパラメータがおかしくなっているのでしょうか。

screenshot_from_2015-06-22 20 32 15

@YoheiKakiuchi
Copy link
Member

このコミットでセンサが傾いてつけるようになっています。
fkanehiro/openhrp3@c83e28f

以下のyamlのimuの設定にいい感じでrotationを入れると良いのではないかと思います。
https://github.com/start-jsk/rtmros_gazebo/blob/master/hrpsys_gazebo_general/config/SampleRobot.yaml#L122

@robograffitti
Copy link
Member Author

imu_sensor0: {ros_name: 'sample_imu_sensor', link_name: 'WAIST_LINK0', frame_id: 'WAIST_LINK0', translation: [0, 0, 0], rotation: [1, 0, 0, 0]}
のように追記して、rotationの値をいい感じに変えてやるということだと思うのですが、
そもそもopenhrp3の方でこのような変更が加えられたのはどうしてでしょうか。
openhrp3の方でこのままセンサが傾いてついた状態は維持されて、
こっちでrotationを入れてpullreqすればいいですか?

@eisoku9618
Copy link
Member

そもそもopenhrp3の方でこのような変更が加えられたのはどうしてでしょうか。

kfのデバッグ用に僕が加えました.

hrpsys-simulatorでは動いていたので大丈夫だったと思っていましたが…

@eisoku9618
Copy link
Member

歩かないのはstep timeを小さくすると歩くと思います。

https://github.com/start-jsk/rtmros_gazebo/blob/master/hrpsys_gazebo_general/worlds/empty.world#L20

また、ちょっとあれですが、(send *ri* :calibrate-inertia-sensor)したら真っ直ぐになります。

@k-okada
Copy link
Member

k-okada commented Jun 22, 2015

というか,そもそも,imu_sensors_configが有るのはなぜだろう.
センサの情報がcolladaにはいっていて,それがgazeboに取り込まれていれば,
この情報は必要ないと思うんだけど...

◉ Kei Okada

On Tue, Jun 23, 2015 at 12:14 AM, Eisoku Kuroiwa [email protected]
wrote:

歩かないのはstep timeを小さくすると歩くと思います。

https://github.com/start-jsk/rtmros_gazebo/blob/master/hrpsys_gazebo_general/worlds/empty.world#L20

また、ちょっとあれですが、(send ri :calibrate-inertia-sensor)したら真っ直ぐになります。


Reply to this email directly or view it on GitHub
#191 (comment)
.

@YoheiKakiuchi
Copy link
Member

それがgazeboに取り込まれていれば,

これがむずかしそうで,実装されていないです.
ですが,yamlに書いてある情報でimuが一番現実的にできそうではあります.

@k-okada
Copy link
Member

k-okada commented Jun 22, 2015

ふーむ.pr2みるとurdfから spawn_modelしていて,imuがみえているから,簡単そうだけどなぁ.

◉ Kei Okada

2015-06-23 1:24 GMT+09:00 Yohei Kakiuchi [email protected]:

それがgazeboに取り込まれていれば,

これがむずかしそうで,実装されていないです.
ですが,yamlに書いてある情報でimuが一番現実的にできそうではあります.


Reply to this email directly or view it on GitHub
#191 (comment)
.

@robograffitti
Copy link
Member Author

(send ri :calibrate-inertia-sensor) したらまっすぐになりました。
ありがとうございます。とりあえずはこれで動かそうと思います。

@k-okada
Copy link
Member

k-okada commented Jun 24, 2015

本来の趣旨とは違う方向で定着してしまうと良くないのでちゃんと治したいところです.

◉ Kei Okada

2015-06-23 21:38 GMT+09:00 Yoshimaru Tanaka [email protected]:

(send ri :calibrate-inertia-sensor) したらまっすぐになりました。
ありがとうございます。とりあえずはこれで動かそうと思います。


Reply to this email directly or view it on GitHub
#191 (comment)
.

@snozawa
Copy link
Contributor

snozawa commented Jun 24, 2015

これは、collada=>urdfに変換したときにセンサ情報が残らないということでしょうか。
それともurdfにセンサ情報をかいてもgazeboで読み込まれないということでしょうか。

@k-okada
Copy link
Member

k-okada commented Jun 24, 2015

ちゃんとはわかっていませんが,
SampleRobot.yamlにちゃんと書いていないから,手で書いてコミットするは最低限として,
このファイルが自動生成なら部分に位置姿勢がちゃんと入るように直すのがいいのと,
urdfにgazeboのimuがはいっていないなら,それをいれるのがいいでしょう.
という感じです.

◉ Kei Okada

2015-06-24 13:16 GMT+09:00 Shunichi Nozawa [email protected]:

これは、collada=>urdfに変換したときにセンサ情報が残らないということでしょうか。
それともurdfにセンサ情報をかいてもgazeboで読み込まれないということでしょうか。


Reply to this email directly or view it on GitHub
#191 (comment)
.

@snozawa
Copy link
Contributor

snozawa commented Jun 24, 2015

pr2がimuをよんでるのも、とくに(PR2用の)gazeboプラグインがあるのでなくて、
gazebo正規の機能として読み込んでimu値を出してるかんじでしょうか。
であれば特殊なことをrtmros_gazeboでしなくてもでてきそうですね。。。

@k-okada
Copy link
Member

k-okada commented Jun 24, 2015

https://github.com/PR2/pr2_common/blob/indigo-devel/pr2_description/urdf/sensors/microstrain_3dmgx2_imu.urdf.xacro
から
https://github.com/PR2/pr2_common/blob/indigo-devel/pr2_description/urdf/sensors/microstrain_3dmgx2_imu.gazebo.xacro
を読んで
https://github.com/ros-simulation/gazebo_ros_pkgs/blob/jade-devel/gazebo_plugins/src/gazebo_ros_imu.cpp
を使っているように見えます.

◉ Kei Okada

2015-06-24 13:37 GMT+09:00 Shunichi Nozawa [email protected]:

pr2がimuをよんでるのも、とくに(PR2用の)gazeboプラグインがあるのでなくて、
gazebo正規の機能として読み込んでimu値を出してるかんじでしょうか。
であれば特殊なことをrtmros_gazeboでしなくてもでてきそうですね。。。


Reply to this email directly or view it on GitHub
#191 (comment)
.

@YoheiKakiuchi
Copy link
Member

これは、collada=>urdfに変換したときにセンサ情報が残らないということでしょうか。
それともurdfにセンサ情報をかいてもgazeboで読み込まれないということでしょうか。

両方ですね。
前者は、urdfのテキストにはgazeboタグが書けるけど、ROSの側では読み込まないので、collada_to_urdfするときに、ROSで一度読んだ後にcolladaをもう一度パースしてurdfに追加する必要があります。

後者は、現状コードがないです。
PR2の方式を使うとROSのトピックとしてimuが出てしまいます。
現状では、IOBPlugin(gazebo) <-> iob(hrpsys)のみが通信することでhrpsysとgazeboの同期をとっています。
ですが、複数個imuがなければそれほど問題なくIOBPluginで書けると思います。

@YoheiKakiuchi
Copy link
Member

start-jsk/rtmros_tutorials#332 で修正しています。

ちょっと間違ったことを書きました。
urdfにxacroでセンサ情報を書いています。iobで読むためにsensor_id(センサの順番)だけimu_sensors_configに書く必要があります。

@snozawa
Copy link
Contributor

snozawa commented Jun 24, 2015

前者は、urdfのテキストにはgazeboタグが書けるけど、ROSの側では読み込まないので、collada_to_urdfするときに、ROSで一度読んだ後にcolladaをもう一度パースしてurdfに追加する必要があります。

前者の問題は、collada_to_urdfがセンサを読み込まないので、その生成物にもセンサが含まれない、そのためあとで別途追加が必要ということでしょうか。

後者は、現状コードがないです。
PR2の方式を使うとROSのトピックとしてimuが出てしまいます。
現状では、IOBPlugin(gazebo) <-> iob(hrpsys)のみが通信することでhrpsysとgazeboの同期をとっています。
ですが、複数個imuがなければそれほど問題なくIOBPluginで書けると思います。

/imuがトピックとしてでていると何が問題でしょうか。
remapすれば良い?

start-jsk/rtmros_tutorials#332 で修正しています。

こちらも手書きでかかずに、正規のmodel conversionの流れでできるものでしょうか。
また、idはいれられそうじゃないでしょうか?
http://wiki.ros.org/urdf/XML/sensor/proposals
http://sdformat.org/spec?ver=1.5&elem=sensor

あとこれに関連して、もしかしたらcolladaを経由する方法をやめてurdfを経由するように色々書き換えたら
それで良い気もしてきています。

robograffitti pushed a commit to robograffitti/rtmros_gazebo that referenced this issue Jun 24, 2015
…raight-end-coords

Modify staro robotiq straight end coords
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

No branches or pull requests

5 participants