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

Controller requiring robot_description parameter does not load anymore after switch to loading robot_description from topic #1450

Closed
firesurfer opened this issue Mar 19, 2024 · 4 comments
Labels

Comments

@firesurfer
Copy link
Contributor

Describe the bug

Please see the related issue in the cartesian controllers repository: fzi-forschungszentrum-informatik/cartesian_controllers#179

I recently (tried) to switch from passing the robot_description as a parameter to loading it from the corresponding topic.
The cartesian controllers apparently depend on obtaining the robot_description from the respective parameter.

This issue is rather a question than a pure bug report: What is the intended way of accessing the robot description inside a controller?

a) Via the robot_description parameter
b) Load it yourself from the topic
c) A different way (e.g. ros2control provides the URDF ready to use to the controller?)

To Reproduce

  1. Configure ros2control to load the robot_description from the topic
  2. Try to load any of the cartesian controllers

Environment (please complete the following information):

@firesurfer firesurfer added the bug label Mar 19, 2024
@firesurfer firesurfer changed the title Controller requiring robot_description parameter does not load anymore after switch to loading robot_description from topc Controller requiring robot_description parameter does not load anymore after switch to loading robot_description from topic Mar 19, 2024
@christophfroehlich
Copy link
Contributor

christophfroehlich commented Mar 19, 2024

The URDF is already a member of the base class:

@firesurfer
Copy link
Contributor Author

Ah great to know. Do you also plan on backporting this to iron/humble ?

@christophfroehlich
Copy link
Contributor

No, I don't think so because #1088 would break API (init is not a method used by users, but still at least breaks ABI for controllers.)

We will stop backporting such changes, and motivate users to build the latest rolling version for their distros from source.

@saikishor
Copy link
Member

I agree with @christophfroehlich backporting it will break API and ABI. It's recommended from now on to use the rolling version to use the main features

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

No branches or pull requests

3 participants