Update st, abc, el, and travis.sh to pass travis tests #903

snozawa · 2015-12-13T08:17:42Z

travisのテストが通らないorとおりにくい箇所を直しました。
#900
を含んでいます。
travsiで確認中なので、まだマージしないようよろしくお願いいたします。

…ames and equation numbers.

…Y error checking and check base RPY error between reference and actual.

…e which does not pass samplerobot_auto_balancer.py test.

…tate().command instead of rtm.readDataPort of el joint angle output to keep thread safety.

…install reported in fkanehiro#900 (comment)

k-okada · 2015-12-13T08:20:16Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2456/

Build Log
last 10 lines

[...truncated 3676 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:201)
    at hudson.model.Run.execute(Run.java:1783)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)

Test FAILed.

…or limit checking after version '315.5.0'

k-okada · 2015-12-13T09:04:17Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2457/

Build Log
last 10 lines

[...truncated 3676 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:201)
    at hudson.model.Run.execute(Run.java:1783)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)

Test FAILed.

snozawa · 2015-12-13T10:04:01Z

変更内容についてのメモです。

abcのテストが不十分で、適切にうごいてなくてもテストが通ってましたので、samplerobot_auto_balacer.pyを修正してテストが通らないようにしました(倒れた判定でabsをとるひつようがあった)。
@eisoku9618 さん。上記を直した上で、eisoku9618@7a8bc67 の内容が通ってなかったようなのでrevertしました。
elで、readDataPortでなくて落ちていたので、getActualStateをつかうようにしました。

snozawa · 2015-12-13T21:15:36Z

確認できてtravisも通りましたので、mergeいただけますと助かります

Update st, abc, el, and travis.sh to pass travis tests

k-okada · 2015-12-14T01:05:44Z

.travis.sh

@@ -180,7 +180,8 @@ case $TEST_PACKAGE in
 pkg=$TEST_PACKAGE
 sudo apt-get install -qq -y python-wstool ros-hydro-catkin ros-hydro-mk ros-hydro-rostest ros-hydro-rtmbuild ros-hydro-roslint > /dev/null

- sudo apt-get install -qq -y ros-hydro-pcl-ros ros-hydro-moveit-commander ros-hydro-rqt-robot-dashboard > /dev/null
+ # Add inputting of "N" for mongodb configuration during deb install reported in https://github.com/fkanehiro/hrpsys-base/pull/900#issuecomment-162392884


そうですね．
https://s3.amazonaws.com/archive.travis-ci.org/jobs/95277537/log.txt
mongo dbの部分は実行されてそうでした．

apt-get remove ではなくて，apt-get purgeにしたらいいんじゃないかと思う．
yes N　.. はいろいろ危険なきがする．

#906
で確認中です。

ところでこれはなぜ必要なんでしたっけ。
travisでpre installされて使われているmongodbと、他で使っている（今回はたmたまmoveit-commander）mongodbとでバージョンが違うのでしょうか。

On Mon, Dec 14, 2015 at 12:24 PM, Shunichi Nozawa [email protected]
wrote:

travisでpre
installされて使われているmongodbと、他で使っている（今回はたmたまmoveit-commander）mongodbとでバージョンが違うのでしょうか。

ああ，なるほど．そうかもしれないですね．

◉ Kei Okada

coveralls · 2017-10-30T09:03:29Z

Changes Unknown when pulling 676bac4 on snozawa:update_st_abc_el into ** on fkanehiro:master**.

snozawa added 5 commits December 13, 2015 12:05

[rtc/Stabilizer/Stabilizer.*] Update Stabilizer doc including paper n…

579eb86

…ames and equation numbers.

[sample/SampleRobot/samplerobot_auto_balancer.py] Use abs for Base RP…

e970cf0

…Y error checking and check base RPY error between reference and actual.

[rtc/AutoBalancer/AutoBalancer.cpp] Revert AutoBalancer 7a8bc67 chang…

ad4eb10

…e which does not pass samplerobot_auto_balancer.py test.

[sample/SampleRobot/samplerobot_soft_error_limiter.py] Use getActualS…

f2970b3

…tate().command instead of rtm.readDataPort of el joint angle output to keep thread safety.

[.travis.sh] Add inputting of N for mongodb configuration during deb …

9dad254

…install reported in fkanehiro#900 (comment)

[sample/SampleRobot/samplerobot_soft_error_limiter.py] Check soft err…

676bac4

…or limit checking after version '315.5.0'

fkanehiro added a commit that referenced this pull request Dec 13, 2015

Merge pull request #903 from snozawa/update_st_abc_el

bd0b2a2

Update st, abc, el, and travis.sh to pass travis tests

fkanehiro merged commit bd0b2a2 into fkanehiro:master Dec 13, 2015

k-okada reviewed Dec 14, 2015
View reviewed changes

snozawa mentioned this pull request Dec 14, 2015

Update travis for mongodb #906

Merged

snozawa deleted the update_st_abc_el branch December 14, 2015 13:33

snozawa mentioned this pull request Dec 14, 2015

Update Stabilizer doc including paper names and equation numbers. #900

Closed

eisoku9618 mentioned this pull request Dec 18, 2015

Fix fixed coords again #917

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update st, abc, el, and travis.sh to pass travis tests #903

Update st, abc, el, and travis.sh to pass travis tests #903

snozawa commented Dec 13, 2015

k-okada commented Dec 13, 2015

k-okada commented Dec 13, 2015

snozawa commented Dec 13, 2015

snozawa commented Dec 13, 2015

k-okada Dec 14, 2015

snozawa Dec 14, 2015

k-okada Dec 14, 2015

coveralls commented Oct 30, 2017 •

edited

Loading

Update st, abc, el, and travis.sh to pass travis tests #903

Update st, abc, el, and travis.sh to pass travis tests #903

Conversation

snozawa commented Dec 13, 2015

k-okada commented Dec 13, 2015

k-okada commented Dec 13, 2015

snozawa commented Dec 13, 2015

snozawa commented Dec 13, 2015

k-okada Dec 14, 2015

Choose a reason for hiding this comment

snozawa Dec 14, 2015

Choose a reason for hiding this comment

k-okada Dec 14, 2015

Choose a reason for hiding this comment

coveralls commented Oct 30, 2017 • edited Loading

coveralls commented Oct 30, 2017 •

edited

Loading