[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker. #49

130s · 2016-10-07T05:07:05Z

@kdhansen Thanks for the comment. The suggested change uses industrial_ci that has a capability to run Kinetic-Xenial on Docker.

Travis CI failing for this PR should be a separate issue (I won't have time to take a deeper look until next weekend, sorry).

UPDATE1. Sorry for much noise. Finally test passed...A lot of commits are squashed and now ready for the review and merge.

To summarize, what this PR does (which has diverted from the original title):

3585ff0 Add a config file for Travis for kinetic branch, using industrial_ci package. ROS prerelease test is also run as an option.
d78a042 OpenCV3 on Xenial adjustment.
73cef7f Enable runtime test using bag files. Only for rgbd
b6f9062 hztest didn't seem to test well with the bag files available on download.ros.org/face_detector/. So use publishtest.

With rgbd launch faces are sometimes detected, so publishtest passes.

kdhansen · 2016-10-17T15:48:50Z

Travis seems to stop at the Before_script with no further explanation.

130s · 2017-02-15T20:57:04Z

Sorry for delay. With industrial_ci much updated since the last run, now it seems to be working, and seems to be detecting something wrong on Xenial-Kinetic.

https://travis-ci.org/wg-perception/people/jobs/202012035#L3204

[ERROR] [1487191552.700384696]: Cascade file /usr/share/opencv/haarcascades/haarcascade_frontalface_alt.xml doesn't exist.

, which I assume prevents roslaunch from running and thus tests fail.

On my Xenial machine, the file in question haarcascade_frontalface_alt.xml is found at the following dir, which is different from what's expected above.

/opt/ros/kinetic/share/OpenCV-3.1.0-dev/haarcascades/haarcascade_frontalface_alt.xml

130s · 2017-02-15T21:22:07Z

The issue in the above comment is now successfully worked around by 4daa0af, which isn't a good solution at all though.

Now, still seems that either roslaunch isn't running nor something else is happening.

mathias-luedtke · 2017-02-15T21:56:24Z

ecc3dfc introduces invalid XML syntax:

* WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest">
    <group unless="$(arg expected_success)">
      <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/>
      <param name="hz" value="0.0"/>
    </group>
    <group if="$(arg expected_success)">
      <node args="-r 0.5 $(find face_detector)/test/face_detector_withface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/>
      <param name="hz" value="30.0"/>
      <param name="hzerror" value="29.9"/>
    </group>
    
    <param name="topic" value="face_detector/people_tracker_measurements_array"/>
    <param name="test_duration" value="15.0"/>
  </test>

And something is wrong with the target dependencies:

Warnings << people_tracking_filter:cmake /root/catkin_ws/logs/people_tracking_filter/build.cmake.000.log
CMake Warning (dev) at CMakeLists.txt:61 (add_dependencies):
  Policy CMP0046 is not set: Error on non-existent dependency in
  add_dependencies.  Run "cmake --help-policy CMP0046" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  The dependency target "people_msgs_gencpp" of target "people_tracker" does
  not exist.
This warning is for project developers.  Use -Wno-dev to suppress it.

Not sure if this fixes the tests..

… confusing. For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in [this PR](wg-perception/people#49): ``` * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest"> <group unless="$(arg expected_success)"> <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/> <param name="hz" value="0.0"/> </group> ``` This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact tag, I didn't try to read it because I was told the issue is with <node>, not <test>. With this PR the message becomes a bit verbose but hope the readers get better idea.

… confusing. For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in this [PR](wg-perception/people#49)): ``` * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest"> <group unless="$(arg expected_success)"> <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/> <param name="hz" value="0.0"/> </group> ``` This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact <test> tag, I didn't try to read it because I was told the issue is with <node>, not <test>. With this PR the message becomes a bit verbose but hope the readers get better idea.

130s · 2017-02-18T00:15:06Z

Sorry for much noise. Finally test passed...A lot of commits are squashed and now ready for the review and merge.

To summarize, what this PR does (which has diverted from the original title):

3585ff0 Add a config file for Travis for kinetic branch, using industrial_ci package.
d78a042 OpenCV3 on Xenial adjustment.
73cef7f Enable runtime test using bag files. Only for rgbd
b6f9062 hztest didn't seem to test well with the bag files available on download.ros.org/face_detector/. So use publishtest.

With rgbd launch faces are sometimes detected, so publishtest passes.

…V 3.1.0.

[face_detector][test common] Fix warning in .launch format. Fix namespace in testcases. Increase retry. .bag-based perceptional tests are hard to deterministic, so it makes sense to do this. Set param use_sim_time. [face_detector][test] hztest can be used for measureing no publishment. hzerror is not needed in that case. [face_detector][test] Specify test name. Pass all args. [face_detector][test] pass_all_args might require all args kept out of include tag. [face_detector][test] Specify test-name. [face_detector][test] Disable all tests other than rgbd. I couldn't figure out how to use launches other than face_detector.rgbdlaunch, after hours of trial on local machine. I leave that up to someone else. [face_detector][test] Detection using rgbd works with this config. [face_detector][test] Shorter test duration (bag files are less than 7 seconds). [face_detector][test] Shorter test_duration. We only need a second or so as long as we can confirm the designated topic is being published. [face_detector][test] rosbag play with clock and loop option.

Since the publish rate of the detected face varies every second because one of the persons in the bag file is moving, [publishtest](http://wiki.ros.org/rostest/Nodes#publishtest) should suit the best in this case. [face_detector][test common] Enable substitution in rosparam tag.

… confusing. For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in this [PR](wg-perception/people#49)): ``` * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest"> <group unless="$(arg expected_success)"> <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/> <param name="hz" value="0.0"/> </group> ``` This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact <test> tag, I didn't try to read it because I was told the issue is with <node>, not <test>. With this PR the message becomes a bit verbose but hope the readers get better idea.

130s · 2017-02-28T04:36:30Z

Ping any admins. This is ready for review (and merge).

… confusing. For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in this [PR](wg-perception/people#49)): ``` * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest"> <group unless="$(arg expected_success)"> <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/> <param name="hz" value="0.0"/> </group> ``` This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact <test> tag, I didn't try to read it because I was told the issue is with <node>, not <test>. With this PR the message becomes a bit verbose but hope the readers get better idea.

130s · 2017-03-27T20:25:25Z

Ping any admins. This is ready for review (and merge).

Thanks!

… confusing. For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in this [PR](wg-perception/people#49)): ``` * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest"> <group unless="$(arg expected_success)"> <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/> <param name="hz" value="0.0"/> </group> ``` This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact <test> tag, I didn't try to read it because I was told the issue is with <node>, not <test>. With this PR the message becomes a bit verbose but hope the readers get better idea.

130s force-pushed the k/update_travis branch from eb366e9 to 57ccd3e Compare October 7, 2016 08:50

kdhansen mentioned this pull request Oct 8, 2016

leg_detector kinetic rtree issue #44

Closed

130s force-pushed the k/update_travis branch 3 times, most recently from 9f11976 to bcf4f17 Compare October 15, 2016 03:02

130s force-pushed the k/update_travis branch from bcf4f17 to ecc3dfc Compare February 15, 2017 20:37

130s mentioned this pull request Feb 15, 2017

non-prerelease test fails with '/root/.ros/X/rosout.log': No such file or directoryError opening rosout log ros-industrial/industrial_ci#121

Closed

130s mentioned this pull request Feb 16, 2017

[roslaunch] Indicating <node> tag when it is actually a <test> tag is confusing. ros/ros_comm#989

Closed

130s mentioned this pull request Feb 16, 2017

Better not hardcode haarcascades config file location #52

Open

130s force-pushed the k/update_travis branch from 23508e0 to 4ddd428 Compare February 16, 2017 05:11

130s force-pushed the k/update_travis branch 10 times, most recently from d18780c to 4be4152 Compare February 17, 2017 20:09

130s force-pushed the k/update_travis branch 3 times, most recently from d157761 to b6f9062 Compare February 18, 2017 00:07

130s added 5 commits February 17, 2017 16:17

[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker.

52e8547

[test] Missing dependency.

e2b25b1

[face_detector] an obtuse fix to hardcoded path for Xenial with OpenC…

9d274b3

…V 3.1.0.

130s force-pushed the k/update_travis branch from b6f9062 to a7ad0e8 Compare February 18, 2017 00:17

130s mentioned this pull request Feb 18, 2017

Fix for the leg_detector opencv3 bug #48

Closed

kdhansen mentioned this pull request Mar 28, 2017

Update leg_detector to OpenCV3 #53

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker. #49

[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker. #49

130s commented Oct 7, 2016 •

edited

Loading

kdhansen commented Oct 17, 2016

130s commented Feb 15, 2017

130s commented Feb 15, 2017

mathias-luedtke commented Feb 15, 2017

130s commented Feb 18, 2017

130s commented Feb 28, 2017

130s commented Mar 27, 2017

[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker. #49

Are you sure you want to change the base?

[kinetic] Update Travis config to run on Ubuntu 16.04 via Docker. #49

Conversation

130s commented Oct 7, 2016 • edited Loading

kdhansen commented Oct 17, 2016

130s commented Feb 15, 2017

130s commented Feb 15, 2017

mathias-luedtke commented Feb 15, 2017

130s commented Feb 18, 2017

130s commented Feb 28, 2017

130s commented Mar 27, 2017

130s commented Oct 7, 2016 •

edited

Loading