-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat(controllers)!: remove robot description parameter #186
Conversation
a9f1eea
to
289f83f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
technically I don't think this is a breaking change, because trying to set a parameter that doesn't exist fails but doesn't result in an error. And the controller behavior should stay the same whether it gets the robot description from the parameter or from the controller manager. Especially as the parameter was marked internal, removing it also shouldn't change any user experience.
Still, I don't mind it being staged on a breaking branch for now for testing
Hmm true, however we will have to release this as part of v6 because of the base image change IMO. This would not work with the current v2.0.0-jazzy image.
It is marked internal, but downstream controllers might fail to configure if it's not available so that's another reason to call it "breaking" |
Description
This PR removes the
robot_description
parameter of the controller interface because the robot description is already available throughget_robot_description()
of the ros2 control base class.Breaking change and requires new base image, goes into staging branch
Review guidelines
Estimated Time of Review: 5 minutes
Checklist before merging: