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

Get foosteps #939

Merged
merged 5 commits into from
Jan 9, 2016
Merged

Get foosteps #939

merged 5 commits into from
Jan 9, 2016

Conversation

eisoku9618
Copy link
Contributor

@snozawa さん

go-posの中でfootstepを作っている関数を外から任意のタイミングで呼べるようになっていたので,
x y thが与えられてたときのfootstepを取得できるようにしましたが,よろしいでしょうか?

@k-okada
Copy link
Contributor

k-okada commented Jan 8, 2016

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/2541/
Test PASSed.

@snozawa
Copy link
Contributor

snozawa commented Jan 8, 2016

go_pos_param_2_xx関数がconst関数になると良いですね.

…cerService_impl.cpp, AutoBalancerService_impl.h] add getFootstepsSequence function
@k-okada
Copy link
Contributor

k-okada commented Jan 8, 2016

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/2543/
Test PASSed.

@k-okada
Copy link
Contributor

k-okada commented Jan 8, 2016

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/2544/
Test PASSed.

@eisoku9618
Copy link
Contributor Author

go_pos_param_2_xx関数がconst関数になると良いですね.

go_pos_param_2_xxの中でconstじゃなくしている要素が大まかには

  • 並進速度をメンバ変数に代入している
  • メンバ変数footstep_nodes_listを上書きしている

の2つありました.

まず,前者はメンバ変数にする必要がなさそうでしたので引数として渡すように変更しました.
次に,go_pos_param_2_xxの中で純粋にfootstep_nodes_listを作る部分をgo_pos_param_2_xx_coreという関数に分離し,それをconst関数としました.

外からfootstepを要求する関数では,go_pos_param_2_xx_coreの方を呼ぶようにして,const関数であることもコメントしておきました.

ご確認よろしくお願いします.

@snozawa
Copy link
Contributor

snozawa commented Jan 9, 2016

LGTM.
travistとおりましたのでご確認いただけますと助かります。

fkanehiro added a commit that referenced this pull request Jan 9, 2016
@fkanehiro fkanehiro merged commit 96fbab2 into fkanehiro:master Jan 9, 2016
@eisoku9618 eisoku9618 deleted the get-foosteps branch January 9, 2016 06:33
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