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

Add setWrench function for SequencePlayer. Checked with 315.1.10 compati... #420

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Dec 9, 2014

SequencePlayerに一個のエンドエフェクタの目標力・モーメントをセットするsetWrenchを追加しました。
#395
からsetWrenchの部分のみ抜き出してコミットです。

SequencePlayerのidlが変更になりますが、最後尾追加なのと、

  • 315.1.10やこの一個前のバージョンのhrpsys-base
  • このコミットを適用したIDLから作ったもの(テストしたのはhrpsys_ros_bridge越しにeuslispから通信)

でテストして、通信できていることを確認していますが、@k-okadaさん、これでOKでしょうか。
(通信できている=setWrench以外のsetJointAnglesなどのidlにあるサービスポート関数が、エラーなく実行できる)

@k-okada
Copy link
Contributor

k-okada commented Dec 9, 2014

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

@snozawa
Copy link
Contributor Author

snozawa commented Dec 11, 2014

コメントアウトされた箇所を削除する等、微修正しました。
また、
#419
のSampleRobotのSequencePlayerも通ることを確認しました。

あとは、travisが通るようになればOKでしょうか。

@k-okada
Copy link
Contributor

k-okada commented Dec 15, 2014

@130s さん,見ておいてください.トラビス通っているから大丈夫と,なりそうです.

@snozawa
Copy link
Contributor Author

snozawa commented Dec 15, 2014

@k-okadaさん
実はhrpsys-baseのtravis.shがおえてないですが、

  • 315.1.9のhrpsys-baseとhrpsys-baseから生成したROSBridgeと通信する

をチェックしてますでしょうか?

@130s
Copy link
Contributor

130s commented Dec 15, 2014

315.1.9のhrpsys-baseとhrpsys-baseから生成したROSBridgeと通信する

travis の以下のあたり でチェックをして頂いているようです.

        else
            echo "
            #
            # check rtmros_common compiled on newer version of hrpsys works with deb version of hrpsys
            # [hrpsys:old] <-> [rtmros_common:new] + [hrpsys:new]
            "
            :
            wstool set hrpsys http://github.com/start-jsk/hrpsys -v 315.1.9 --git -y

@snozawa
Copy link
Contributor Author

snozawa commented Dec 16, 2014

@130s さん
実際にseqなどのRTCにアクセスしてテストしている箇所はどこになりますでしょうか。
手元でも一応確認はしましたが、travisで通っているのでbackward compatibilityが成り立ってますとなっているか知りたいと思います。

@130s
Copy link
Contributor

130s commented Dec 17, 2014

間違っているやも知れませんが,RTC のチェックは下流テストに依存して行なっているというのが回答になるでしょうか.
ここで下流テストを行なっていて,少なくともテスト対象の下流 package の一つとして指定されている rtmros_hironx ではシミュレーションで RTC を起動しています.

@k-okada
Copy link
Contributor

k-okada commented Dec 17, 2014

backward compatibilityのチェックは

else

がポイントで,
wstool set rtmros_common http://github.com/start-jsk/rtmros_common --git -y

でlatestなrtmros_common, rtmros_hironx, rtmros_nextageを持ち込みつつ,
sed -i "1imacro(dummy_install)\nmessage(\"install(\${ARGN})\")\nendmacro()" src/hrpsys/CMakeLists.txt

で,いま確認しているPRを使ってコンパイルはするけどinstallはしないということをして,
wstool set hrpsys http://github.com/start-jsk/hrpsys -v 315.1.9 --git -y

でインストール用のhrpsysを315.1.9になるようにして持ってきてコンパイルして

source install/setup.bash

で,rtmros_commonは新しいIDLでコンパイルされたけど,テストコードが動くときのhrpsysは315.1.9 になる
というものを作ってあるつもりです.

全く怪しいのでダブルチェックしてみてください.テスト中にhrpsysのversion確認していると思うので,それで
古くなっているか,とか,idlから.msgをつくっているところは,最新のものができているか,などが確認ポイント
でしょうか.

◉ Kei Okada

2014-12-17 9:51 GMT+09:00 Isaac I.Y. Saito notifications@github.com:

間違っているやも知れませんが,RTC のチェックは下流テストに依存して行なっているというのが回答になるでしょうか.
ここで

find $pkg_path/test -iname "*.test" -print0 | xargs -0 -n1 rostest || export EXIT_STATUS=$?;

下流テストを行なっていて,少なくともテスト対象の下流 package の一つとして指定されている rtmros_hironx
- TEST_TYPE=work_with_downstream TEST_PACKAGE=hrpsys-ros-bridge

ではシミュレーションで RTC を起動しています.


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

@snozawa
Copy link
Contributor Author

snozawa commented Dec 19, 2014

なるほど、古いhrpsys-baseとlastest hrpsys-baseからできた諸々のチェックをしてるんですね。

全く怪しいのでダブルチェックしてみてください.テスト中にhrpsysのversion確認していると思うので,それで 古くなっているか,とか,idlから.msgをつくっているところは,最新のものができているか,などが確認ポイント でしょうか.

travisのログもみてみました。

とくにTEST_TYPE=work_with_315_1_10 TEST_PACKAGE=hironx-ros-bridge
TEST_TYPE=work_with_315_1_10 TEST_PACKAGE=hrpsys-ros-bridgeとで、
hrpsys-base側のhrpsys_config.pyの出力に、latestだと出るはずのversion番号がでてなかったので、古いものが使われてると思います。
また、srvなどの生成をみると、このPRで追加したsetWrenchがあるので、最新rtmros_commonなどが使われています。

ということで、古いhrpsys-baseと新しいrtmros_xxの通信を行っていて、それでテストが通っているのが確認できました。
(hrpsys-baeが古いため、hrpsys-baseで動いている相手自体のバージョンがわからないですが、hrpsys_config.pyのバージョンは古いのは確認できます)

@k-okada
Copy link
Contributor

k-okada commented Dec 21, 2014

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

@snozawa snozawa mentioned this pull request Dec 22, 2014
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