Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Mecanum Drive Controller (backport #512) #1376

Draft
wants to merge 7 commits into
base: humble
Choose a base branch
from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Nov 19, 2024

Depends on ros-controls/control_msgs#79


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

(cherry picked from commit 79358e3)
@soham2560
Copy link

are there any estimates on when this might be done?

or on how much work there is, I can try to pick it up if I get the time and if no-one else is working on it

@bmagyar
Copy link
Member

bmagyar commented Mar 6, 2025

@soham2560 yes please! Create PR(s) targeting this backport branch please.

There were a few API changes in controller interface between humble and Jazzy. Those definitely need fixing for the build to pass.

@christophfroehlich
Copy link
Contributor

you can always compile the rolling version of the ros2_control stack on humble, see "development version" on https://control.ros.org/humble/doc/getting_started/getting_started.html#building-from-source

Comment on lines +46 to +48
Used when the controller is not in chained mode (``in_chained_mode == false``).

- <controller_name>/reference [geometry_msgs/msg/TwistStamped]
Copy link
Contributor

@christophfroehlich christophfroehlich Mar 7, 2025

Choose a reason for hiding this comment

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

@soham2560 can you please fix this section? (you can add a review comment here, and I'll commit it).

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.

4 participants