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 testGaitGenerator and fix bug of overritable-footstep-index-offset = 0 #1139

Merged
merged 17 commits into from
May 22, 2017

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented May 22, 2017

Thanks for @kyawawa

GaitGeneratorで歩行するときにoverritable-footstep-index-offset = 0にする(=遊脚の着地位置を変更可能にする)と

  • 変更時に足先姿勢の補間が不連続になる
  • swing support foot mid coords (swing->supportまで連続的に遷移する代表座標)の補間が不連続になる(@kyawawa報告)

の2点があったので、修正して、テストを追加しました。

よろしくお願いします

snozawa added 17 commits May 21, 2017 16:10
…rn generation parts as functions. This will not change the program behavior.
…of swing foot rot (for example, foot step modification for rotation)
…of foot midcoords (for example, foot step modification for updown and rotation). This bug is originally reported by @kyawawa.
@snozawa snozawa closed this May 22, 2017
@snozawa snozawa reopened this May 22, 2017
@snozawa
Copy link
Contributor Author

snozawa commented May 22, 2017

修正内容について補足します
@kyawawa, @YutaKojio

主な修正は、遊脚機中に着地位置を変更したときに、付随するパラメタの

の2点についてです。
どちらも原因は同じで、

  • 今まで
    startからgoalまでを補間する比率を、xx_ratio_interpolatorという名前で0から1まで連続的にかえてたが、goalが変わると破綻した
  • 本PR
    補間のgoalを常にかえて(setGoal)、現在からgoalまでの補間するようにしたので、goalが変わってもOK

とかえてます。
@kyawawaと相談した修正方針と異なる点は、interpolatorを使ってる、GaitGeneratorの中でだけで閉じて計算している、などです。

また、一つStableRTCのSequencePlayerのdimensionを持ってくる箇所が
doubleのところがintであるべきだったので、修正しました
b11a633

このPRがmergeされたあとできる確認手段は

testGaitGenerator --test18 --use-graph-append true # imagemagickがつかえたら
testGaitGenerator --test18

などです

@YutaKojio
Copy link
Contributor

今まで
startからgoalまでを補間する比率を、xx_ratio_interpolatorという名前で0から1まで連続的にかえてたが、goalが変わると破綻した

今までは,高さ方向でなく水平方向のみの着地位置変更でもおかしくなっていたのでしょうか?

@snozawa
Copy link
Contributor Author

snozawa commented May 22, 2017

今までは,高さ方向でなく水平方向のみの着地位置変更でもおかしくなっていたのでしょうか?

そうですね。
水平のみ着地位置補正のときは、足先遊脚姿勢は影響なかったと思いますが、
swing support foot midcoordsの不具合は起きてて、偶然バグを踏まなかったくらいのかんじだと思います。
この座標系の値のうち、

  • 位置XYは、重心軌道などがうわがいてくれるので、ふつうにしてると問題が露見しないです。けど、この座標系にeusからおくったコンフィギュレーション(SequencePlayerからおくったもの)をfixしてたりするので、手まで入れたstartAuoBalancerすると問題おこってたと思います。
  • 回転成分と位置Z成分は、明治的に変更しなければ露見しなかったと思います

なかんじなので、足のみStartAutoBalancerくらいだと露見しないけど、それ以外の使い方で露見するイメージです

@YutaKojio
Copy link
Contributor

なるほど.修正ありがとうございます.

@snozawa snozawa closed this May 22, 2017
@snozawa snozawa reopened this May 22, 2017
@snozawa snozawa closed this May 22, 2017
@snozawa snozawa reopened this May 22, 2017
@snozawa
Copy link
Contributor Author

snozawa commented May 22, 2017

travis何度かrestartして通りました。
お手すきの時で構いませんので、ご確認いただけますと幸いです

@fkanehiro fkanehiro merged commit 800ec62 into fkanehiro:master May 22, 2017
@snozawa snozawa deleted the update_gg_test_fix_bug branch May 22, 2017 09:37
@snozawa
Copy link
Contributor Author

snozawa commented May 22, 2017

ありがとうございます。

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