rqt-jtc: Make continuous sending opt-in and add Send Once button - #2466
Open
petertheprocess wants to merge 1 commit into
Open
rqt-jtc: Make continuous sending opt-in and add Send Once button#2466petertheprocess wants to merge 1 commit into
petertheprocess wants to merge 1 commit into
Conversation
Fixes ros-controls#1579 The rqt_joint_trajectory_controller repeatedly republishes a trajectory with a fixed time_from_start (~1.0 s) at 10 Hz as soon as the controller is enabled. Because the joint has already moved between successive sends, each new trajectory is re-interpolated from a slightly different current position, producing jitter in the reference and erratic joint movement on both hardware and simulation. Changes: - Enabling (arming) a controller no longer starts continuous sending automatically. A new "continuous send" checkbox (off by default) must be ticked to republish commands periodically, restoring the old behaviour on demand. - Add a "Send Once" button that publishes a single trajectory on demand, independently of the continuous-send checkbox. - Add an adjustable "start time (ms)" spin box that overrides time_from_start when > 0; 0 keeps the auto-computed duration from joint velocity limits and speed scaling. - Add an adjustable "pub freq (Hz)" spin box for the continuous-send rate. - Refactor trajectory building into _build_trajectory_msg() shared by both the timer callback and Send Once. - Document the new controls in userdoc.rst and add a CHANGELOG entry.
github-actions
Bot
requested review from
Juliaj,
MarqRazz,
mhubii,
moriarty and
saikishor
July 8, 2026 12:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The rqt_joint_trajectory_controller repeatedly republishes a trajectory with a big time_from_start (~1.0 s) at 10 Hz as soon as the controller is enabled. Because the joint has already moved between successive sends, each new trajectory is re-interpolated from a slightly different current position, producing jitter in the reference and erratic joint movement on both hardware and simulation.
Changes:
Description
Fixes #1579
Is this user-facing behavior change?
Yes,new components are added in the GUI allowing user to set time_from_start + publish freqency mannuly.

Did you use Generative AI?
Yes, I am using GLM5.2 within opencode.
Additional Information