Skip to content

Fix gripper node naming to use robot_type for correct topic/action prefixes#206

Open
Benkess wants to merge 1 commit intofrankarobotics:humblefrom
Benkess:fix/gripper-node-name-robot-type
Open

Fix gripper node naming to use robot_type for correct topic/action prefixes#206
Benkess wants to merge 1 commit intofrankarobotics:humblefrom
Benkess:fix/gripper-node-name-robot-type

Conversation

@Benkess
Copy link
Copy Markdown

@Benkess Benkess commented Feb 7, 2026

Summary

This pull request fixes franka_gripper/launch/gripper.launch.py to follow the intended gripper node naming convention by deriving the node name from robot_type.

The gripper node name is now generated as <robot_type>_gripper instead of being hard-coded to franka_gripper.

Motivation

The gripper node publishes topics and actions using private (~) names, so the node name determines the effective topic/action prefix. After the introduction of robot namespacing, the gripper node name was no longer aligned with robot_type, which caused:

  • Gripper topics/actions to be published under franka_gripper/*
  • MoveIt controller configs and joint-state aggregation to expect fr3_gripper/*
  • Gripper joint states not being merged into /joint_states, causing MoveIt to report the gripper as always closed

Changes

  • Gripper node name changed from franka_gripper to <robot_type>_gripper
  • Restores correct topic/action prefixes (e.g. fr3_gripper/... for FR3)
  • Fixes MoveIt joint-state aggregation so gripper states correctly appear in /joint_states

…efixes

Signed-off-by: Ben Kessler <BenPKessler@gmail.com>
BarisYazici pushed a commit that referenced this pull request Mar 2, 2026
…r deps

Merge in MOCTRL/franka_ros2 from fix-missed-mobile-sensor-deps to jazzy

* commit '57bc9031361b4dfaa1daff053c34568a1ebcf7ac':
  docs: Added docu link to the 'mobile_sensors' package
  add: docu for franka_mobile_sensors package
  fix: franka_mobile_sensors formatting and linting issues
  fix: The docker build was missing the mobile_sensor deps
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.

1 participant