Skip to content

Allow disabling robot_state_publisher - #771

Open
galou wants to merge 1 commit into
luxonis:humblefrom
galou:opt_robot_state_publisher
Open

Allow disabling robot_state_publisher#771
galou wants to merge 1 commit into
luxonis:humblefrom
galou:opt_robot_state_publisher

Conversation

@galou

@galou galou commented Jan 22, 2026

Copy link
Copy Markdown

The launch argument is description.enable, defaulting to true for backward compatibility.

Overview

Author: Gaël Écorchard

Issue

Issue link (if present):
Issue description:
Related PRs

Changes

ROS distro: humble
Add the description.enable launch argument to camera.launch.py.

Testing

Hardware used: irrelevant
Depthai library version: irrelevant

Visuals from testing

Add screenshots/gifs/videos from RVIZ or other visualizers demonstrating the effect of the changes when applicable.

RViz will not show the robot model because it is not published.

The launch argument is `description.enable`, defaulting to true for
backward compatibility.

Signed-off-by: Gaël Écorchard <gael@km-robotics.cz>
@galou

galou commented Jan 23, 2026

Copy link
Copy Markdown
Author

I saw afterward that there's the camera_as_part_of_a_robot.launch.py but I'm actually using pointcloud.launch.py, which uses camera.launch.py. Those using camera.launch.py directly and facing the same issue can rather switch to camera_as_part_of_a_robot.launch.py.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new launch argument to optionally skip launching the URDF/robot_state_publisher portion of the camera bringup, preserving existing behavior by default.

Changes:

  • Introduce description.enable launch argument (default "true").
  • Gate the URDF include (urdf_launch.py) behind IfCondition(description.enable).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants