Skip to content

Update PublishVelocityForceCommand to use unstamped twist/wrench#638

Merged
marioprats merged 2 commits into
mainfrom
update-publish-velocity-force-command
May 19, 2026
Merged

Update PublishVelocityForceCommand to use unstamped twist/wrench#638
marioprats merged 2 commits into
mainfrom
update-publish-velocity-force-command

Conversation

@marioprats
Copy link
Copy Markdown

example_ws changes after https://github.com/PickNikRobotics/moveit_pro/pull/19088, to avoid deprecation warnings because of the stamped ports.

Requires https://github.com/PickNikRobotics/moveit_pro/pull/19088 to be merged

@marioprats marioprats requested a review from shaur-k May 18, 2026 09:47
Comment on lines +213 to +225
<Control ID="Sequence" name="Sequence">
<Action
ID="ExtractTwist"
name="ExtractTwist"
twist="{twist}"
twist_stamped="{desired_twist}"
/>
<Action
ID="ExtractWrench"
name="ExtractWrench"
wrench="{wrench}"
wrench_stamped="{desired_wrench}"
/>
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.

same q here if we can just use the already existing Unpack Behaviors

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.

Switched to use Unpack Behaviors

@marioprats marioprats force-pushed the update-publish-velocity-force-command branch from bbda44f to 9ff6820 Compare May 18, 2026 13:30
@marioprats marioprats requested a review from shaur-k May 18, 2026 13:37
@marioprats marioprats merged commit cf20db4 into main May 19, 2026
8 of 10 checks passed
@marioprats marioprats deleted the update-publish-velocity-force-command branch May 19, 2026 14:38
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.

2 participants