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

Overwrite current footstep #916

Merged
merged 6 commits into from
Dec 19, 2015
Merged

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Dec 18, 2015

@YutaKojio @eisoku9618
現在の遊脚の着地位置からovewriteするための変更を加えました。
現状、velocityモードで動作するのを確認しています。
samplerobot_auto_balancer.pyの追加サンプルがそれになります。
ただ、なんとなく遊脚軌道がおかしいかも?

よろしくお願いします。

…ssing and add comments. This should not change behaviour.
…enerator.h] Enable to set overwritable_footstep_index_offset.
…r.h] Enable to set overwritable_footstep_index_offset = 0.
@k-okada
Copy link
Contributor

k-okada commented Dec 18, 2015

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

@YutaKojio
Copy link
Contributor

現在の遊脚の着地位置を上書きながら歩行できることを確認しました.
ありがとうございます.

遊脚軌道がおかしい件については,https://github.com/fkanehiro/hrpsys-base/blob/master/rtc/AutoBalancer/GaitGenerator.cpp#L545
if ( lcg.get_lcg_count() > static_cast<size_t>(footstep_nodes_list[lcg.get_footstep_index()][0].step_time/dt * 0.3) - 1 && lcg.get_lcg_count() < static_cast<size_t>(footstep_nodes_list[lcg.get_footstep_index()][0].step_time/dt * 0.7) - 1 ) {
と書き換えるとよくなったようにみえます.

@snozawa
Copy link
Contributor Author

snozawa commented Dec 18, 2015

コミットを更新しました。
並進の遊脚軌道がおかしいのは直しました。
旋回はまだおかしいです。

@k-okada
Copy link
Contributor

k-okada commented Dec 18, 2015

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

@snozawa
Copy link
Contributor Author

snozawa commented Dec 19, 2015

こちらですが、使用には問題ないのでmergeしていただけますとたすかります

fkanehiro added a commit that referenced this pull request Dec 19, 2015
@fkanehiro fkanehiro merged commit 89b61a2 into fkanehiro:master Dec 19, 2015
@snozawa snozawa deleted the fix_overwrite_fs branch December 19, 2015 02:52
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 2c753fe on snozawa:fix_overwrite_fs into ** on fkanehiro:master**.

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.

5 participants