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

Getting to a flexible components management from URDF in /robot_description topic #1045

Open
destogl opened this issue Jun 5, 2023 · 3 comments

Comments

@destogl
Copy link
Member

destogl commented Jun 5, 2023

This description of issues is related to the following PRs:

To solve this knot, I figured out the following 4 steps that will be done in separate PRs to make them manageable:

  1. Rename parameters for setting components in unconfigured and inactive state after CM is started. This feature does not break anything, and it should be back ported to humble as well. See: Enable setting of initial state in HW compoments #1046
  2. Enable loading components from topic if they are not loaded from parameter. Only the first callback will be considered to avoid any complexity with managing potential changes of components in URDF (fixing [CM] Use robot_description topic instead of parameter and don't crash on empty URDF 🦿 #940).
  3. Enable not_loaded components after start. This needs some changes in RM (ABI breaking) that will separate some things better. Enable non-loaded hardware components after start of CM. #1049
  4. Enable updates of /robot_description and components in URDF. The default behavior will be that if a new component exists it will be added to the system, but none component will be removed if it doesn't exist in URDF. User will be notified about this and they can manually remove the component.
@christophfroehlich
Copy link
Contributor

Ad #940: Can it be that it was forgotten to update the documentation

robot_description (mandatory; string)
String with the URDF string as robot description.
This is usually result of the parsed description files by ``xacro`` command.

according to this deprecation warning?
RCLCPP_WARN(
get_logger(),
"[Deprecated] Passing the robot description parameter directly to the control_manager node "
"is deprecated. Use '~/robot_description' topic from 'robot_state_publisher' instead.");
init_resource_manager(robot_description);

@destogl
Copy link
Member Author

destogl commented Jul 18, 2023

Yes, you are correct, docs has to be updated.

@fmauch
Copy link
Contributor

fmauch commented Feb 16, 2024

I can try to update the docs.

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