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 to use compile under catkin_make_isolated #363

Merged
merged 10 commits into from
Oct 15, 2014
Merged

update to use compile under catkin_make_isolated #363

merged 10 commits into from
Oct 15, 2014

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Oct 11, 2014

No description provided.

@k-okada
Copy link
Contributor Author

k-okada commented Oct 11, 2014

Test PASSed.
Refer to this link for build results: http:https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1647/

@k-okada
Copy link
Contributor Author

k-okada commented Oct 11, 2014

Test PASSed.
Refer to this link for build results: http:https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1649/

@k-okada
Copy link
Contributor Author

k-okada commented Oct 15, 2014

Test PASSed.
Refer to this link for build results: http:https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1652/

fkanehiro added a commit that referenced this pull request Oct 15, 2014
update to use compile under catkin_make_isolated
@fkanehiro fkanehiro merged commit 46c09e2 into fkanehiro:master Oct 15, 2014
@k-okada k-okada deleted the use_catkin branch October 15, 2014 08:09
@k-okada
Copy link
Contributor Author

k-okada commented Jul 19, 2015

ふるいテストは関数レベルでテストしようとしていましたが,
いまはrtcとしてlaunchしてその入出力で見るようにしました.
関数レベル動いている!=RTCとして動いている
で最終的に必要なのはRTCとして動いていることが重要なのでこちらを優先しました.

関数レベルでテストできればいいですが,そもそもseqあたりの関数の仕様があやふやだから
そこからですかね.

@snozawa
Copy link
Contributor

snozawa commented Jul 19, 2015

関数レベルでテストできればいいですが,そもそもseqあたりの関数の仕様があやふやだから
そこからですかね.

そうですね。
seqも部分のサンプル・テストを今追加しようと思いまして、そういえばtest_SequenePlayerはどこいったかと探していたところでした。

また、Unstable RTCもtravisのテストをいくつか追加しようと思い、初歩的な質問になりますが
travisのテストのhttps://travis-ci.org/fkanehiro/hrpsys-base/builds/71618859
TEST_PACKAGE=hrpsysTEST_PACKAGE=hrpsys-baseとの違いはなにでしょうか。
https://github.com/fkanehiro/hrpsys-base/blob/master/.travis.sh#L164のような記述もありました)

追加するとすれば、unstable RTCのテストも関数単位でのテストは時間はかからなさそうですが、
hrpsys-simulator+ hrpsys_configを立ち上げてsample/SampleRobot/xxx.pyのようなテストをしようとすると
ちょっと実行時間がかかるので、新たに
https://github.com/fkanehiro/hrpsys-base/blob/master/.travis.yml#L15
にunstbaleのテストの部分を追加するのが良いでしょうか。

@k-okada
Copy link
Contributor Author

k-okada commented Jul 19, 2015

https://github.com/fkanehiro/hrpsys-base/blob/master/test/test-samplerobot-sequence.py
みたい追加したらいいんだと思います.matrixについかするのは面倒だと思います.

@k-okada
Copy link
Contributor Author

k-okada commented Jul 19, 2015

hrpsys とあるのは,ROSとは関係なくcmake/make
でテストできる範囲です.なので関数レベルのテストができたらここで対応するはずです(make testsとかは追加する必要が有ると思う)

hrpsys-base以降はcatkinとかrostestとかつかってチェックします.hrpsys-baseはhrpsysが動くか,hrpsys-toolsはhrpsys-toolsのパッケージが動くか,,,などなどを確認します.
#724

◉ Kei Okada

2015-07-19 16:04 GMT+09:00 Shunichi Nozawa [email protected]:

関数レベルでテストできればいいですが,そもそもseqあたりの関数の仕様があやふやだから
そこからですかね.

そうですね。
seqも部分のサンプル・テストを今追加しようと思いまして、そういえばtest_SequenePlayerはどこいったかと探していたところでした。

また、Unstable RTCもtravisのテストをいくつか追加しようと思い、初歩的な質問になりますが
travisのテストのhttps://travis-ci.org/fkanehiro/hrpsys-base/builds/71618859
TEST_PACKAGE=hrpsysとTEST_PACKAGE=hrpsys-baseとの違いはなにでしょうか。

https://github.com/fkanehiro/hrpsys-base/blob/master/.travis.sh#L164のような記述もありました)

追加するとすれば、unstable RTCのテストも関数単位でのテストは時間はかからなさそうですが、
hrpsys-simulator+
hrpsys_configを立ち上げてsample/SampleRobot/xxx.pyのようなテストをしようとすると
ちょっと実行時間がかかるので、新たに
https://github.com/fkanehiro/hrpsys-base/blob/master/.travis.yml#L15
にunstbaleのテストの部分を追加するのが良いでしょうか。


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

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.

None yet

3 participants