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 footstep planner sample #643

Merged
merged 6 commits into from
Nov 9, 2016

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Nov 8, 2016

@YoheiKakiuchi さん
footstep plannerをeusから呼ぶサンプルを更新して、テストなどを追加しました。
各コミットは以下です。

  • optimistic_footstep_planner.launchでuse_obstacle_modelできるようにしました(デフォルトfalse == 挙動かわらない)。またrvizでobstacleてんぐんを表示するようにしました
  • use_lazy_perceptionのパラメタは、USE_PERCEPTION引数を使うようにしました(デフォルトtrue == 挙動かわらない)
  • footstep client sampleの関数を、utilsに移動させて、関数名をながくてわかりやすくしました
  • (jskeusglのsample_possionのかわり)簡易ですが、引数のbodiesから点群をつくるようにしました(内包をチェックしてる)
  • footstep_graph.cppで、use_obstacle_modelがtrueでかつ点群サイズが0のときにセグフォしていたので、点群サイズが0のときにはisCollideをぬけて干渉ナシにするようにしました。
  • testを追加して、cmakeでよばれるようにしました。

…tic_footstep_planner.launch] Enable use_obstacle_model and display obstacle model as point cloud.
…aunch] Check arg for USE_PERCEPTION for use_lazy_perception
…footstep-planner-client-sample.l] Move footstep action client utility to util.l and rename functions.
…se if use obstacle model is true and no obstacles are specified (point cloud size = 0).
@snozawa snozawa force-pushed the update_footstep_planner_sample branch 3 times, most recently from ae1f138 to 206c9eb Compare November 9, 2016 04:25
@snozawa
Copy link
Contributor Author

snozawa commented Nov 9, 2016

testを追加して、cmakeでよばれるようにしました。

testは追加しましたが、諸々確認中ですので
jsk-ros-pkg/geneus#51
一旦、まだcmakeでaddrostestしてtravis上で呼ばれるようにはしなくしました。

@snozawa snozawa merged commit 5028c90 into jsk-ros-pkg:master Nov 9, 2016
@snozawa snozawa deleted the update_footstep_planner_sample branch November 9, 2016 05:14
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.

1 participant