Skip to content

Modified controller config and launch file for ros2 jazzy#3

Open
mitsav01 wants to merge 17 commits into
code-iai:iai_dualarmfrom
mitsav01:fix/jazzy
Open

Modified controller config and launch file for ros2 jazzy#3
mitsav01 wants to merge 17 commits into
code-iai:iai_dualarmfrom
mitsav01:fix/jazzy

Conversation

@mitsav01

@mitsav01 mitsav01 commented Jun 14, 2026

Copy link
Copy Markdown

Description

This PR improves the robotiq_description package by introducing a more flexible and user-friendly launch workflow for both single and dual Robotiq gripper configurations.

Additionally, this incorporates previous enhancements to the repository for Extended State Feedback, which introduces Effort Reporting and Object Detection capabilities to provide deeper telemetry and status monitoring from the grippers.

The package now supports:

  • Robotiq 2F-85 and 2F-140 grippers
  • Single and dual gripper deployments
  • Automatic controller manager namespace handling
  • Automatic serial port detection for connected Robotiq devices
  • Simplified launch interface with configurable launch arguments

In addition, comprehensive documentation has been added to explain launch options, hardware configuration, controller architecture, and common usage examples.


Changes

Launch System

  • Added unified robotiq_control.launch.py to launch single or multiple grippers with custom controller config file
  • Added automatic USB device detection for Robotiq/Espressif adapters

ROS 2 Control

  • Added support for dual controller managers:
      * /left_grippper/controller_manager
      * /right_gripper/controller_manager
  • Added automatic joint prefixing for dual setups
  • Improved controller loading and namespace isolation

Controllers

  • Added robotiq_controllers_85.yaml
  • Added robotiq_controllers_140.yaml
  • Updated controller configuration to use ROS 2 Jazzy-compatible controller types (replacing deprecated controller interfaces)

Documentation

  • Added complete README documentation including:
      * Quick start guide
      * Launch examples
      * Available launch arguments

Testing

Tested the following configurations:

  • Single Robotiq 2F-140 (fake hardware)
  • Single Robotiq 2F-85 (fake hardware)
  • Dual Robotiq 2F-85 (fake hardware)
  • Dual controller manager namespace creation
  • Controller spawning and activation
  • Automatic serial port detection
  • Single Robotiq 2f-140 (real hardware)
  • Dual Robotiq 2f-85 (real hardware)

@mitsav01 mitsav01 changed the base branch from iai_dualarm to main June 16, 2026 16:33
@mitsav01 mitsav01 changed the base branch from main to iai_dualarm June 16, 2026 16:33
@mitsav01 mitsav01 marked this pull request as ready for review June 17, 2026 12:26

@J-Schaefer J-Schaefer 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.

Some smaller things. Looks good otherwise.

Comment thread robotiq_description/config/robotiq_controllers_140.yaml Outdated
Comment thread documentation/requirements.txt Outdated
@@ -0,0 +1,3 @@
openai>=1.0.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why do we need this package?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I had rebased the my branch fix/jazzy to 'code-iai:main` and it came from there. However, it is not neccessary for the functionality we need from this package

Comment thread documentation/agent.py Outdated
@@ -0,0 +1,205 @@
import os

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is this file only for foundation-based development? Should we push this to a ROS2 package repo? What do you think @maltehue ?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

yeah i think we can remove it

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.

7 participants