Skip to content

Write DOUT 0-7 to CO8-CO15 on a CB7 (backport #2047) - #2058

Merged
urfeex merged 2 commits into
kiltedfrom
mergify/bp/kilted/pr-2047
Nov 6, 2026
Merged

urfeex merged 2 commits into
kiltedfrom
mergify/bp/kilted/pr-2047

Conversation

@mergify

@mergify mergify Bot commented Nov 4, 2026

Copy link
Copy Markdown

CB7 doesn't have standard digital inputs anymore. Instead, indices 0-7 go to CO8-CO15, while indices 8-15 keep going to CO0-CO7.

This requires URCL master at the moment, since it uses the accessors added in UniversalRobots/Universal_Robots_Client_Library@1c78952

Also, this has only been manually tested. Ideally, we would test things in CI against a CB7 robot, which could be added after #1910 has been merged. Hence, this is currently a draft, though the functionality is there and will probably stay like that.


This is an automatic backport of pull request #2047 done by Mergify.

CB7 doesn't have standard digital inputs anymore. Instead, indices 0-7
go to CO8-CO15, while indices 8-15 keep going to CO0-CO7.

(cherry picked from commit 1fb9f96)

# Conflicts:
#	ur_robot_driver/doc/usage/utility_controllers.rst
#	ur_robot_driver/include/ur_robot_driver/hardware_interface.hpp
@mergify

mergify Bot commented Nov 4, 2026

Copy link
Copy Markdown
Author

Cherry-pick of 1fb9f96 has failed:

On branch mergify/bp/kilted/pr-2047
Your branch is up to date with 'origin/kilted'.

You are currently cherry-picking commit 1fb9f96.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   ur_robot_driver/src/hardware_interface.cpp

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   ur_robot_driver/doc/usage/utility_controllers.rst
	both modified:   ur_robot_driver/include/ur_robot_driver/hardware_interface.hpp

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@cursor

cursor Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes real-time digital output routing on CB7 hardware; wrong detection would misdrive I/O, though legacy control boxes keep the previous path.

Overview
CB7 control boxes no longer expose classic standard digital outputs; this PR keeps the existing ROS GPIO index layout while sending writes to the correct pins on the robot.

On connect, the hardware interface now reads control_box_type_ from URCL. In async I/O, commands on standard_digital_output_cmd_ indices 0–7 use sendConfigurableDigitalOutput(i + 8) on CB7 (CO8–CO15) and still use sendStandardDigitalOutput on older boxes. Indices 8–15 and tool outputs are unchanged.

New utility_controllers.rst documents utility controllers (joint state, speed scaling, GPIO/I/O, tool contact, TCP pose, configuration, gravity) and documents the CB7 vs CB3/CB5 digital output index mapping for set_io and io_states.

Reviewed by Cursor Bugbot for commit c59c8fa. Bugbot is set up for automated code reviews on this repo. Configure here.

The target branch uses default true
@urfeex urfeex added enhancement New feature or request and removed conflicts labels Nov 5, 2026
@mergify

mergify Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Author

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@urfeex
urfeex merged commit 182ae7a into kilted Nov 6, 2026
8 of 10 checks passed
@urfeex
urfeex deleted the mergify/bp/kilted/pr-2047 branch November 6, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request kilted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant