-
Notifications
You must be signed in to change notification settings - Fork 544
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
Re-implement dynamic_reconfigure #129
Comments
Henning, I believe rqt_reconfigure is available now for dashing: We have not implemented the reconfigure callbacks as of yet, but here is one PR to do so for AMCL. |
Thanks, I wasn't aware of this one.
Great to have this as an example. |
Henning, there is also a PR in progress to create a ParameterEventsSubscriber class in rclcpp: If you have time please take a look at it and provide any feedback if it is useful for MoveIt2. |
With https://github.com/PickNikRobotics/generate_parameter_library being applied, this issue can be closed. |
We need to re-implement runtime updates of node parameters using
dynamic_reconfigure
.A rather straight-forward approach would use parameter event callbacks as shown in this example:
https://github.com/ros2/tutorials/blob/master/rclcpp_tutorials/src/parameters/parameter_events.cpp
The only downside I see is that we don't automatically register "configurable nodes" and there is no
reconfigure_gui
anymore. Also, we will need to update the client sides or at least provide migration steps.One example where
dynamic_reconfigure
is used in the ompl planner interface: https://github.com/ros-planning/moveit2/blob/cd27142259e13d4b34072bdc6e823288cc234d99/moveit_planners/ompl/ompl_interface/src/ompl_planner_manager.cpp#L262@mkhansen-intel how did you handle this with navigation2?
The text was updated successfully, but these errors were encountered: