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

[ROS2_controller] Add community controllers in "Controllers for Mobile Robots" list on ROS2_control doc #1098

Open
robotcopper opened this issue Apr 11, 2024 · 3 comments

Comments

@robotcopper
Copy link

Feature description:

Hi,
is it possible to add at least a community controllers list to the Controllers for Mobile Robots list already existing on control.ros.org?

Objectif description:

Even include controllers created by the community in the official documentation so that @mateusmenezes95's three-wheeled omnidirectional robot controller can be added.To avoid the difficulty of finding these controllers.

Feature alternative:

Include @mateusmenezes95's three-wheeled omnidirectional robot controller in Controllers for Mobile Robots list in the official documentation.

@christophfroehlich christophfroehlich transferred this issue from ros-controls/ros2_control Apr 11, 2024
@christophfroehlich
Copy link
Contributor

Hi! Yes, this would be possible. But maybe @mateusmenezes95 wants to contribute to the ros2_controllers repository and add his controllers here?
I see that there is already some documentation: I would be glad to add the omni-directional robots to the docs, which I started here (reviews are very welcome!)

@mateusmenezes95
Copy link

Hello @christophfroehlich

Merge the omnidirectional_controller into the official ros2_controllera repo would be a pleasure. However, I am concerned with two points. First, this controller has yet to be tested on a real robot; I only tested it in simulation. Thus, I don't know its maturity level in embedding in a real robot. Secondly, I need to implement some unit tests for the controller, but I won't be able to until the second semester of this year. I'm pretty busy right now finishing up my master's degree, but I'll get on it as soon as I can.

Thus, I think the link is enough by now, but in the second semester, we can try to merge it into the official repository.

@christophfroehlich
Copy link
Contributor

Nice to hear. I see that there are already some tests there, there is not a lot missing to catch up with the controllers from our repo here (could be more though). And some work needs to be done to migrate the controller to the rolling branch, because we don't merge new features in non-rolling versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants