Skip to content

[RQT-JTC] limits from jtc controlled joints#1146

Merged
bmagyar merged 4 commits into
ros-controls:masterfrom
delihus:rqt-jtc-limits-from-jtc-controlled-joints
Jun 5, 2024
Merged

[RQT-JTC] limits from jtc controlled joints#1146
bmagyar merged 4 commits into
ros-controls:masterfrom
delihus:rqt-jtc-limits-from-jtc-controlled-joints

Conversation

@delihus

@delihus delihus commented May 23, 2024

Copy link
Copy Markdown
Contributor

The further explanation of this issue is here #1131

delihus and others added 3 commits January 11, 2023 14:19
changed default value from `odom` -> `base_link`
Signed-off-by: Jakub Delicat <jakub.delicat@husarion.com>

@christophfroehlich christophfroehlich left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for extracting this change.
I tested it with genuine continuous and revolute joints as well as with a revolute joint missing lower/upper limits -> LGTM

@bmagyar bmagyar merged commit 0ae3e42 into ros-controls:master Jun 5, 2024
@christophfroehlich

Copy link
Copy Markdown
Member

@Mergifyio backport iron humble

@mergify

mergify Bot commented Aug 26, 2024

Copy link
Copy Markdown
Contributor

backport iron humble

✅ Backports have been created

Details

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.

3 participants