ros / geometry

Packages for common geometric calculations including the ROS transform library, "tf". Also includes ROS bindings for "bullet" physics engine and "kdl" kinematics/dynamics package.
172 stars 274 forks source link

Format 2 and build_export_depend orocos kdl #210

Closed sloretz closed 4 years ago

sloretz commented 4 years ago

Switch to format to in order to <build_export_depend>liborocos-kdl-dev</build_export_depend> in packages which export the dependency in their CMakelists.txt

Should fix this error

00:02:14.275 CMake Error at /opt/ros/noetic/share/eigen_conversions/cmake/eigen_conversionsConfig.cmake:113 (message):
00:02:14.275   Project 'eigen_conversions' specifies
00:02:14.275   '/usr/share/orocos_kdl/cmake/../../../include' as an include dir, which is
00:02:14.275   not found.  It does neither exist as an absolute directory nor in
00:02:14.275   '${prefix}//usr/share/orocos_kdl/cmake/../../../include'.  Check the
00:02:14.275   website 'http://ros.org/wiki/eigen_conversions' for information and
00:02:14.275   consider reporting the problem.
00:02:14.275 Call Stack (most recent call first):
00:02:14.275   /opt/ros/noetic/share/catkin/cmake/catkinConfig.cmake:76 (find_package)
00:02:14.275   CMakeLists.txt:4 (find_package)

in the dept_image_proc binary jobs

http://build.ros.org/view/Nbin_uF64/job/Nbin_uF64__depth_image_proc__ubuntu_focal_amd64__binary/2

sloretz commented 4 years ago

Test failure in tf.lookupTransform_ring45. It seems unlikely to be related to this PR, though I don't see it in the latest geometry dev job.

00:08:50.697 [ RUN      ] tf.lookupTransform_ring45
00:08:50.697 Warning: TF_REPEATED_DATA ignoring data with redundant timestamp for frame b at time 10.000000 according to authority authority
00:08:50.697          at line 278 in /tmp/binarydeb/ros-noetic-tf2-0.7.1/src/buffer_core.cpp
00:08:50.697 /tmp/ws/src/geometry/tf/test/tf_unittest.cpp:226: Failure
00:08:50.697 Value of: mTR.setTransform(ts, "authority")
00:08:50.697   Actual: false
00:08:50.697 Expected: true
00:08:50.697 Warning: TF_REPEATED_DATA ignoring data with redundant timestamp for frame b at time 0.010000 according to authority authority
00:08:50.697          at line 278 in /tmp/binarydeb/ros-noetic-tf2-0.7.1/src/buffer_core.cpp
00:08:50.697 /tmp/ws/src/geometry/tf/test/tf_unittest.cpp:226: Failure
00:08:50.697 Value of: mTR.setTransform(ts, "authority")
00:08:50.697   Actual: false
00:08:50.697 Expected: true
00:08:50.698 Warning: TF_REPEATED_DATA ignoring data with redundant timestamp for frame b at time 1.000000 according to authority authority
00:08:50.698          at line 278 in /tmp/binarydeb/ros-noetic-tf2-0.7.1/src/buffer_core.cpp
00:08:50.698 /tmp/ws/src/geometry/tf/test/tf_unittest.cpp:226: Failure
00:08:50.698 Value of: mTR.setTransform(ts, "authority")
00:08:50.698   Actual: false
00:08:50.698 Expected: true
00:08:50.698 unknown file: Failure
00:08:50.698 C++ exception with description "Could not find a connection between 'b' and 'a' because they are not part of the same tree.Tf has two or more unconnected trees." thrown in the test body.
00:08:50.698 [  FAILED  ] tf.lookupTransform_ring45 (1 ms)
tfoote commented 4 years ago

The test failure looks like it might be related to: https://github.com/ros/geometry2/pull/459/commits/6e505b1cfc16af860e970f010873afa9902af36f

sloretz commented 4 years ago

The test failure looks like it might be related to: ros/geometry2@6e505b1

Could be. Definitely unrelated to this PR; I also get the test failure locally using the noetic-devel branch.