Skip to content

Add params frequency_hz, create_socat_tty, ip_adress & port#14

Merged
macmacal merged 8 commits intohumble-develfrom
feature/param_frequency_hz
Aug 18, 2025
Merged

Add params frequency_hz, create_socat_tty, ip_adress & port#14
macmacal merged 8 commits intohumble-develfrom
feature/param_frequency_hz

Conversation

@macmacal
Copy link

@macmacal macmacal commented Aug 12, 2025

Description

Motivation and context

  • Fix the issue with robotiq_hande_driver and UR's RTDE protocol.

Related PRs

How has this been tested?

  • Tested manually on the Aegis robot, on the geonosis PC, with the whole aegis_ros stack.

Checklist

  • All TODOs in the code have been resolved or linked to a proper issue.
  • Code has been (auto)formatted.
  • Documentation (e.g., README, CHANGELOG, Wiki) has been updated.
  • All automated checks have passed.

Clickup task: 869972nfu

@macmacal macmacal changed the title Add param frequency_hz Added params frequency_hz, ip_adress Aug 14, 2025
@macmacal macmacal changed the title Added params frequency_hz, ip_adress Added params frequency_hz, create_socat_tty, ip_adress & port Aug 14, 2025
Copy link

@sivral sivral left a comment

Choose a reason for hiding this comment

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

Please update the documentation.

@macmacal macmacal changed the title Added params frequency_hz, create_socat_tty, ip_adress & port Add params frequency_hz, create_socat_tty, ip_adress & port Aug 17, 2025
Copy link

@slimasziom slimasziom left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@macmacal macmacal merged commit b687bc5 into humble-devel Aug 18, 2025
1 check passed
@macmacal macmacal deleted the feature/param_frequency_hz branch August 18, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Config/params/args related enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants