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

:go-posなどが失敗したらnilを返す,成功したらnil以外を必ず返すようにしたい #171

Open
furushchev opened this issue Sep 12, 2015 · 4 comments

Comments

@furushchev
Copy link
Member

テストコードはどう書けばいいでしょうか。

  • 成功するときはnil以外の返り値があることをassert
  • 失敗するときはnilが返ることをassert
  • 失敗させるには:go-posなどでgoalを送っているaction serverに何かを送って中断させる?
@furushchev furushchev changed the title :go-posなどが失敗したらnilを返す, :go-posなどが失敗したらnilを返す,成功したらnil以外を必ず返すようにしたい Sep 12, 2015
@furushchev
Copy link
Member Author

ref #164

@k-okada
Copy link
Member

k-okada commented Sep 12, 2015

なるほど.これって実機/gazebo無しのkinematics simulator 環境でテストかけるのかな?
とりあえず
https://github.com/jsk-ros-pkg/jsk_pr2eus/pull/163/files
の説明(https://github.com/jsk-ros-pkg/jsk_pr2eus/pull/163/files#diff-15f6920b924513f6027d36b361f85bbdR826)のところに ,希望する振る舞いを書いてみましょう.

Cc: @snozawa

@snozawa
Copy link
Contributor

snozawa commented Sep 12, 2015

実はこのあたりがrobot-interface.lにはいってなくて
#163
で追加になるので今までよくわかってなかったのですが、
go-posが失敗するというのはどいうった状況のことでしょうか。

またこれに限らず、指令値送ることに関係するメソッド(:angle-vector, angle-vector-sequence, go-pos, :wait-interpolation...)は
今まで基本的には返り値をとってませんでしたが、これらも同様に
うまくいってたらnil以外、なんらかの問題があればnilをかえす、というのでよいようにも思います。
(もしくは、うまくいったらtで失敗したらエラーフラグに対応したシンボルをかえす)

@k-okada
Copy link
Member

k-okada commented Sep 12, 2015

pr2のgo-posはmobe_base経由だから、障害物があればそこでとまってゴールまで到達しないのです

2015年9月12日土曜日、Shunichi [email protected]さんは書きました:

実はこのあたりがrobot-interface.lにはいってなくて
#163 #163
で追加になるので今までよくわかってなかったのですが、
go-posが失敗するというのはどいうった状況のことでしょうか。

またこれに限らず、指令値送ることに関係するメソッド(:angle-vector, angle-vector-sequence, go-pos,
:wait-interpolation...)は
今まで基本的には返り値をとってませんでしたが、これらも同様に
うまくいってたらnil以外、なんらかの問題があればnilをかえす、というのでよいようにも思います。
(もしくは、うまくいったらtで失敗したらエラーフラグに対応したシンボルをかえす)


Reply to this email directly or view it on GitHub
#171 (comment)
.

◉ Kei Okada

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

No branches or pull requests

3 participants