Skip to content

Commit 6bf539b

Browse files
fix: take dummy objects' height into calculation when locating their Z position (#4195)
* Update node.cpp fix: consider dummy objects' height when locating its Z position * style(pre-commit): autofix --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 3b0ef6a commit 6bf539b

File tree

1 file changed

+2
-2
lines changed
  • simulator/dummy_perception_publisher/src

1 file changed

+2
-2
lines changed

simulator/dummy_perception_publisher/src/node.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,7 @@ void DummyPerceptionPublisherNode::objectCallback(
359359
ros_map2base_link = tf_buffer_.lookupTransform(
360360
"map", "base_link", rclcpp::Time(0), rclcpp::Duration::from_seconds(0.5));
361361
object.initial_state.pose_covariance.pose.position.z =
362-
ros_map2base_link.transform.translation.z;
362+
ros_map2base_link.transform.translation.z + 0.5 * object.shape.dimensions.z;
363363
} catch (tf2::TransformException & ex) {
364364
RCLCPP_WARN_SKIPFIRST_THROTTLE(get_logger(), *get_clock(), 5000, "%s", ex.what());
365365
return;
@@ -406,7 +406,7 @@ void DummyPerceptionPublisherNode::objectCallback(
406406
ros_map2base_link = tf_buffer_.lookupTransform(
407407
"map", "base_link", rclcpp::Time(0), rclcpp::Duration::from_seconds(0.5));
408408
objects_.at(i).initial_state.pose_covariance.pose.position.z =
409-
ros_map2base_link.transform.translation.z;
409+
ros_map2base_link.transform.translation.z + 0.5 * objects_.at(i).shape.dimensions.z;
410410
} catch (tf2::TransformException & ex) {
411411
RCLCPP_WARN_SKIPFIRST_THROTTLE(get_logger(), *get_clock(), 5000, "%s", ex.what());
412412
return;

0 commit comments

Comments
 (0)