From 2798a1691ce1ee067da89c19e786abe512ca6853 Mon Sep 17 00:00:00 2001 From: Wim Meeussen Date: Thu, 24 Mar 2011 17:46:27 -0700 Subject: [PATCH] Clean up code and get regression tests working --- .../robot_state_publisher.h | 5 +- .../src/robot_state_publisher.cpp | 15 +- robot_state_publisher/test/pr2.urdf | 958 +++++++----------- .../test/test_publisher.launch | 3 +- 4 files changed, 367 insertions(+), 614 deletions(-) diff --git a/robot_state_publisher/include/robot_state_publisher/robot_state_publisher.h b/robot_state_publisher/include/robot_state_publisher/robot_state_publisher.h index 0355c13..9ea834c 100644 --- a/robot_state_publisher/include/robot_state_publisher/robot_state_publisher.h +++ b/robot_state_publisher/include/robot_state_publisher/robot_state_publisher.h @@ -72,9 +72,8 @@ public: /** Publish transforms to tf * \param joint_positions A map of joint names and joint positions. * \param time The time at which the joint positions were recorded - * returns true on success; return false when the robot model is empty or not all the joints in the robot model are specified in the joint map. */ - bool publishTransforms(const std::map& joint_positions, const ros::Time& time); + void publishTransforms(const std::map& joint_positions, const ros::Time& time); void publishFixedTransforms(); private: @@ -82,7 +81,7 @@ private: std::map segments_, segments_fixed_; - tf::TransformBroadcaster tf_broadcaster_, tf_broadcaster_fixed_; + tf::TransformBroadcaster tf_broadcaster_; }; diff --git a/robot_state_publisher/src/robot_state_publisher.cpp b/robot_state_publisher/src/robot_state_publisher.cpp index 6202394..d26927a 100644 --- a/robot_state_publisher/src/robot_state_publisher.cpp +++ b/robot_state_publisher/src/robot_state_publisher.cpp @@ -63,11 +63,11 @@ namespace robot_state_publisher{ SegmentPair s(children[i]->second.segment, root, child.getName()); if (child.getJoint().getType() == KDL::Joint::None){ segments_fixed_.insert(make_pair(child.getJoint().getName(), s)); - ROS_INFO("Adding fixed segment from %s to %s", root.c_str(), child.getName().c_str()); + ROS_DEBUG("Adding fixed segment from %s to %s", root.c_str(), child.getName().c_str()); } else{ segments_.insert(make_pair(child.getJoint().getName(), s)); - ROS_INFO("Adding moving segment from %s to %s", root.c_str(), child.getName().c_str()); + ROS_DEBUG("Adding moving segment from %s to %s", root.c_str(), child.getName().c_str()); } addChildren(children[i]); } @@ -75,10 +75,10 @@ namespace robot_state_publisher{ // publish moving transforms - bool RobotStatePublisher::publishTransforms(const map& joint_positions, const Time& time) + void RobotStatePublisher::publishTransforms(const map& joint_positions, const Time& time) { + ROS_DEBUG("Publishing transforms for moving joints"); std::vector tf_transforms; - tf::StampedTransform tf_transform; tf_transform.stamp_ = time; @@ -92,18 +92,15 @@ namespace robot_state_publisher{ tf_transforms.push_back(tf_transform); } } - tf_broadcaster_.sendTransform(tf_transforms); - - return true; } // publish fixed transforms void RobotStatePublisher::publishFixedTransforms() { + ROS_DEBUG("Publishing transforms for moving joints"); std::vector tf_transforms; - tf::StampedTransform tf_transform; tf_transform.stamp_ = ros::Time::now()+ros::Duration(0.5); // future publish by 0.5 seconds @@ -114,7 +111,7 @@ namespace robot_state_publisher{ tf_transform.child_frame_id_ = seg->second.tip; tf_transforms.push_back(tf_transform); } - tf_broadcaster_fixed_.sendTransform(tf_transforms); + tf_broadcaster_.sendTransform(tf_transforms); } } diff --git a/robot_state_publisher/test/pr2.urdf b/robot_state_publisher/test/pr2.urdf index 07308a0..ae3ee20 100644 --- a/robot_state_publisher/test/pr2.urdf +++ b/robot_state_publisher/test/pr2.urdf @@ -1,6 +1,6 @@ - + @@ -14,6 +14,7 @@ + @@ -54,17 +55,23 @@ + + + + + + - + - + - + - + - - + @@ -202,7 +221,7 @@ - + -79.2380952381 @@ -226,7 +245,7 @@ - + @@ -243,9 +262,8 @@ - - + 79.2380952381 @@ -269,7 +287,7 @@ - + @@ -286,9 +304,8 @@ - - + -79.2380952381 @@ -301,8 +318,7 @@ - - + @@ -328,7 +344,7 @@ - + -79.2380952381 @@ -352,7 +368,7 @@ - + @@ -369,9 +385,8 @@ - - + 79.2380952381 @@ -395,7 +410,7 @@ - + @@ -412,9 +427,8 @@ - - + -79.2380952381 @@ -427,8 +441,7 @@ - - + @@ -454,7 +467,7 @@ - + -79.2380952381 @@ -478,7 +491,7 @@ - + @@ -495,9 +508,8 @@ - - + 79.2380952381 @@ -521,7 +533,7 @@ - + @@ -538,9 +550,8 @@ - - + -79.2380952381 @@ -553,8 +564,7 @@ - - + @@ -580,7 +590,7 @@ - + -79.2380952381 @@ -604,7 +614,7 @@ - + @@ -621,9 +631,8 @@ - - + 79.2380952381 @@ -647,7 +656,7 @@ - + @@ -664,9 +673,8 @@ - - + -79.2380952381 @@ -675,7 +683,7 @@ - true + false base_link_geom 100.0 @@ -686,7 +694,9 @@ - + + + @@ -708,8 +718,7 @@ - - + @@ -725,7 +734,7 @@ - + @@ -736,6 +745,34 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + torso_lift_link_geom @@ -747,9 +784,8 @@ - - + -52143.33 @@ -766,19 +802,6 @@ - - - - - - - - - - - - - @@ -799,8 +822,7 @@ - - + @@ -808,28 +830,26 @@ - - - + + + + - + - - - - - + + 6.0 @@ -839,8 +859,7 @@ - - + @@ -848,14 +867,15 @@ - - - + + + + - + @@ -866,10 +886,8 @@ - - - - + + 6.0 @@ -923,12 +941,6 @@ - - - - - - @@ -949,12 +961,6 @@ - - - - - - @@ -975,12 +981,6 @@ - - - - - - @@ -1032,12 +1032,6 @@ - - - - - - @@ -1051,19 +1045,6 @@ - - - - - - - - - - - - - @@ -1083,18 +1064,6 @@ - - - - - - - - - - - - @@ -1145,18 +1114,6 @@ - - - - - - - - - - - - @@ -1216,19 +1173,6 @@ - - - - - - - - - - - - - @@ -1248,18 +1192,6 @@ - - - - - - - - - - - - @@ -1310,18 +1242,6 @@ - - - - - - - - - - - - @@ -1375,13 +1295,66 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + true + 15.0 + stereo_projection_pattern_high_res_red.png + projector_wg6802418_child_frame + stereo_projection_pattern_filter.png + projector_wg6802418_controller/image + projector_wg6802418_controller/projector + 0.785398163397 + 0.4 + 10 + + - - + @@ -1396,7 +1369,7 @@ - + @@ -1419,19 +1392,6 @@ - - - - - - - - - - - - - @@ -1442,7 +1402,7 @@ false -79.9999999086 79.9999999086 - 0.05 + 0.08 10.0 0.01 40 @@ -1457,9 +1417,8 @@ - - - + + -6.05 @@ -1476,8 +1435,7 @@ - - + @@ -1488,7 +1446,7 @@ - + @@ -1505,8 +1463,7 @@ - - + @@ -1521,7 +1478,7 @@ - + @@ -1540,8 +1497,7 @@ - - + @@ -1575,23 +1531,12 @@ - + 32.6525111499 - - r_shoulder_pan_link_geom - 100.0 - - true - 100.0 - r_shoulder_pan_bumper - - - - true @@ -1599,31 +1544,23 @@ - - r_shoulder_lift_link_geom - 100.0 - - true - 100.0 - r_r_shoulder_lift_bumper - - - true - - + + 63.1552452977 + - - + + 61.8948225713 + @@ -1640,7 +1577,7 @@ - + @@ -1652,17 +1589,6 @@ - - r_upper_arm_link_geom - - 100.0 - - true - 100.0 - r_upper_arm_bumper - - - true @@ -1670,8 +1596,7 @@ - - + @@ -1706,7 +1631,7 @@ - + -90.5142857143 @@ -1717,8 +1642,7 @@ - - + @@ -1733,7 +1657,7 @@ - + @@ -1748,24 +1672,13 @@ -1.0 - - r_elbow_flex_link_geom - 100.0 - - true - 100.0 - r_elbow_flex_bumper - - - - true - + -36.167452007 @@ -1785,7 +1698,7 @@ - + @@ -1798,12 +1711,11 @@ - + - + - - + @@ -1817,7 +1729,7 @@ - + @@ -1834,8 +1746,7 @@ - - + @@ -1864,30 +1775,9 @@ true - - r_forearm_link_geom - 100.0 - - true - 100.0 - r_forearm_bumper - - - true - - r_wrist_flex_link_geom - 100.0 - - true - 100.0 - r_wrist_flex_bumper - - - - @@ -1895,22 +1785,12 @@ true - - r_wrist_roll_link_geom - 100.0 - - true - 100.0 - r_wrist_roll_bumper - - - - + @@ -1930,7 +1810,7 @@ - + @@ -1996,7 +1876,7 @@ - + @@ -2026,7 +1906,7 @@ - + @@ -2038,70 +1918,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -2120,7 +1936,7 @@ - + @@ -2150,7 +1966,7 @@ - + @@ -2170,6 +1986,7 @@ true 100.0 + r_gripper_l_finger_link r_gripper_l_finger_bumper @@ -2178,7 +1995,6 @@ - @@ -2194,6 +2010,7 @@ 100.0 true + r_gripper_r_finger_link 100.0 r_gripper_r_finger_bumper @@ -2203,7 +2020,6 @@ - @@ -2219,6 +2035,7 @@ 100.0 true + r_gripper_l_finger_tip_link 100.0 r_gripper_l_finger_tip_bumper @@ -2228,7 +2045,6 @@ - @@ -2244,6 +2060,7 @@ 100.0 true + r_gripper_r_finger_tip_link 100.0 r_gripper_r_finger_tip_bumper @@ -2253,7 +2070,6 @@ - @@ -2281,6 +2097,58 @@ + + true + 0.17126 + 7.7562e-05 + 1.49095e-06 + -9.83385e-06 + 0.000197083 + -3.06125e-06 + 0.000181054 + 0.03598 + 0.0173 + -0.00164 + 0.82991 -0.157 0.790675 + 0 -0 0 + true + false + + + true + 0.17389 + 7.73841e-05 + -2.09309e-06 + -8.36228e-06 + 0.000198474 + 2.4611e-06 + 0.00018107 + 0.03576 + -0.01736 + -0.00095 + 0.82991 -0.219 0.790675 + 0 -0 0 + true + false + + + + + r_gripper_r_parallel_link + r_gripper_palm_link + r_gripper_palm_link + 0 0 -1 + 0.2 + 0.05891 -0.031 0 + + + r_gripper_l_parallel_link + r_gripper_palm_link + r_gripper_palm_link + 0 0 1 + 0.2 + 0.05891 0.031 0 + r_gripper_r_parallel_link r_gripper_r_finger_tip_link @@ -2311,11 +2179,14 @@ + + + true @@ -2324,11 +2195,11 @@ true 100.0 + r_gripper_palm_link r_gripper_palm_bumper - @@ -2343,7 +2214,7 @@ - + - - + @@ -2379,7 +2247,7 @@ - + @@ -2396,8 +2264,7 @@ - - + @@ -2412,7 +2279,7 @@ - + @@ -2431,8 +2298,7 @@ - - + @@ -2466,23 +2332,12 @@ - + 32.6525111499 - - l_shoulder_pan_link_geom - 100.0 - - true - 100.0 - l_shoulder_pan_bumper - - - - true @@ -2490,31 +2345,23 @@ - - l_shoulder_lift_link_geom - 100.0 - - true - 100.0 - l_r_shoulder_lift_bumper - - - true - - + + 63.1552452977 + - - + + 61.8948225713 + @@ -2531,7 +2378,7 @@ - + @@ -2543,17 +2390,6 @@ - - l_upper_arm_link_geom - - 100.0 - - true - 100.0 - l_upper_arm_bumper - - - true @@ -2561,8 +2397,7 @@ - - + @@ -2597,7 +2432,7 @@ - + -90.5142857143 @@ -2608,8 +2443,7 @@ - - + @@ -2624,7 +2458,7 @@ - + @@ -2639,24 +2473,13 @@ -1.0 - - l_elbow_flex_link_geom - 100.0 - - true - 100.0 - l_elbow_flex_bumper - - - - true - + -36.167452007 @@ -2676,7 +2499,7 @@ - + @@ -2689,12 +2512,11 @@ - + - + - - + @@ -2708,7 +2530,7 @@ - + @@ -2725,8 +2547,7 @@ - - + @@ -2755,30 +2576,9 @@ true - - l_forearm_link_geom - 100.0 - - true - 100.0 - l_forearm_bumper - - - true - - l_wrist_flex_link_geom - 100.0 - - true - 100.0 - l_wrist_flex_bumper - - - - @@ -2786,22 +2586,12 @@ true - - l_wrist_roll_link_geom - 100.0 - - true - 100.0 - l_wrist_roll_bumper - - - - + @@ -2821,7 +2611,7 @@ - + @@ -2887,7 +2677,7 @@ - + @@ -2917,7 +2707,7 @@ - + @@ -2929,70 +2719,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -3011,7 +2737,7 @@ - + @@ -3041,7 +2767,7 @@ - + @@ -3061,6 +2787,7 @@ true 100.0 + l_gripper_l_finger_link l_gripper_l_finger_bumper @@ -3069,7 +2796,6 @@ - @@ -3085,6 +2811,7 @@ 100.0 true + l_gripper_r_finger_link 100.0 l_gripper_r_finger_bumper @@ -3094,7 +2821,6 @@ - @@ -3110,6 +2836,7 @@ 100.0 true + l_gripper_l_finger_tip_link 100.0 l_gripper_l_finger_tip_bumper @@ -3119,7 +2846,6 @@ - @@ -3135,6 +2861,7 @@ 100.0 true + l_gripper_r_finger_tip_link 100.0 l_gripper_r_finger_tip_bumper @@ -3144,7 +2871,6 @@ - @@ -3172,6 +2898,58 @@ + + true + 0.17126 + 7.7562e-05 + 1.49095e-06 + -9.83385e-06 + 0.000197083 + -3.06125e-06 + 0.000181054 + 0.03598 + 0.0173 + -0.00164 + 0.82991 0.219 0.790675 + 0 -0 0 + true + false + + + true + 0.17389 + 7.73841e-05 + -2.09309e-06 + -8.36228e-06 + 0.000198474 + 2.4611e-06 + 0.00018107 + 0.03576 + -0.01736 + -0.00095 + 0.82991 0.157 0.790675 + 0 -0 0 + true + false + + + + + l_gripper_r_parallel_link + l_gripper_palm_link + l_gripper_palm_link + 0 0 -1 + 0.2 + 0.05891 -0.031 0 + + + l_gripper_l_parallel_link + l_gripper_palm_link + l_gripper_palm_link + 0 0 1 + 0.2 + 0.05891 0.031 0 + l_gripper_r_parallel_link l_gripper_r_finger_tip_link @@ -3202,11 +2980,14 @@ + + + true @@ -3215,11 +2996,11 @@ true 100.0 + l_gripper_palm_link l_gripper_palm_bumper - @@ -3234,7 +3015,7 @@ - + @@ -3260,18 +3039,6 @@ - - - - - - - - - - - - @@ -3322,18 +3089,6 @@ - - - - - - - - - - - - @@ -3374,3 +3129,4 @@ PR2/Blue + diff --git a/robot_state_publisher/test/test_publisher.launch b/robot_state_publisher/test/test_publisher.launch index 29aa39d..2c954f3 100644 --- a/robot_state_publisher/test/test_publisher.launch +++ b/robot_state_publisher/test/test_publisher.launch @@ -1,5 +1,6 @@ - + +