-
Notifications
You must be signed in to change notification settings - Fork 88
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
base: master
Are you sure you want to change the base?
Conversation
Test PASSed. |
c74897d
to
6d99f3b
Compare
コメントアウトされた箇所を削除する等、微修正しました。 あとは、travisが通るようになればOKでしょうか。 |
@130s さん,見ておいてください.トラビス通っているから大丈夫と,なりそうです. |
@k-okadaさん
をチェックしてますでしょうか? |
travis の以下のあたり でチェックをして頂いているようです.
|
@130s さん |
間違っているやも知れませんが,RTC のチェックは下流テストに依存して行なっているというのが回答になるでしょうか. |
backward compatibilityのチェックは Line 117 in 8a4a68b
がポイントで, Line 125 in 8a4a68b
でlatestなrtmros_common, rtmros_hironx, rtmros_nextageを持ち込みつつ, Line 132 in 8a4a68b
で,いま確認しているPRを使ってコンパイルはするけどinstallはしないということをして, Line 144 in 8a4a68b
でインストール用のhrpsysを315.1.9になるようにして持ってきてコンパイルして Line 156 in 8a4a68b
で,rtmros_commonは新しいIDLでコンパイルされたけど,テストコードが動くときのhrpsysは315.1.9 になる というものを作ってあるつもりです. 全く怪しいのでダブルチェックしてみてください.テスト中にhrpsysのversion確認していると思うので,それで ◉ Kei Okada 2014-12-17 9:51 GMT+09:00 Isaac I.Y. Saito notifications@github.com:
|
なるほど、古いhrpsys-baseとlastest hrpsys-baseからできた諸々のチェックをしてるんですね。
travisのログもみてみました。 とくに ということで、古いhrpsys-baseと新しいrtmros_xxの通信を行っていて、それでテストが通っているのが確認できました。 |
Refer to this link for build results (access rights to CI server needed): |
SequencePlayerに一個のエンドエフェクタの目標力・モーメントをセットするsetWrenchを追加しました。
#395
からsetWrenchの部分のみ抜き出してコミットです。
SequencePlayerのidlが変更になりますが、最後尾追加なのと、
でテストして、通信できていることを確認していますが、@k-okadaさん、これでOKでしょうか。
(通信できている=setWrench以外のsetJointAnglesなどのidlにあるサービスポート関数が、エラーなく実行できる)