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

Update around kf #582

Merged
merged 3 commits into from
Apr 16, 2015
Merged

Update around kf #582

merged 3 commits into from
Apr 16, 2015

Conversation

eisoku9618
Copy link
Contributor

update時に

  • roll:accから求めたroll
  • pitch:accから求めたpitch
  • yaw:0.0

だったので,#550 となってしまっていたところを,

  • roll:accから求めたroll
  • pitch:accから求めたpitch
  • yaw:IMUがついているリンクのyaw

にしました.

@eisoku9618
Copy link
Contributor Author

腰にIMUが付いているロボットで

PR前

bbb

だと,回転しているのに回転していないことになっていて

PR後

aa

だと,しっかりと回転しました

@k-okada
Copy link
Contributor

k-okada commented Apr 16, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1962/
Test PASSed.

@k-okada
Copy link
Contributor

k-okada commented Apr 16, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1964/
Test PASSed.

fkanehiro added a commit that referenced this pull request Apr 16, 2015
@fkanehiro fkanehiro merged commit 1c2cabc into fkanehiro:master Apr 16, 2015
@snozawa
Copy link
Contributor

snozawa commented Apr 18, 2015

@eisoku9618
これのサンプルプログラム、テストプログラムってあります?
なんでも良いのですが、例えば
 この変更の前だと動かず、変更後だと動く
プログラムとかがあるとよさそうです。

@eisoku9618
Copy link
Contributor Author

これのサンプルプログラム、テストプログラムってあります?

加えた変更は

  1. このPR:従来のIMU座標のrpyの推定で,加速度のみからyawを求めるときに0としていた部分を,FKを使った値にした
  2. output the body frame relative to a world reference frame as baseRpyCurr... #586 のPR:従来のkfがstに出していたrpyポートはそのままにした上で,baseRpyCurrentというポートを追加した

で,テストは

#582 (comment)

https://github.com/fkanehiro/hrpsys-base/pull/582/files#diff-09092c2f0677809ee35e50a1e5c56a8bR89 をもとに戻した場合

hoge

https://github.com/fkanehiro/hrpsys-base/pull/582/files#diff-09092c2f0677809ee35e50a1e5c56a8bR89 をもとに戻さずこのPRのようにした場合

image

になります.

これとは別に,サンプルロボットのIMUを腰に移動させてもらいましたが,https://github.com/fkanehiro/openhrp3/pull/56,

上の2つのグラフを比較することで,IMUが腰に付いている場合に腰を回すとyawがおかしくなっているのが治っていることがわかると思います.

@eisoku9618 eisoku9618 deleted the update-around-kf branch July 27, 2015 16:38
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.

4 participants