With #349 we've introduced a new verb, ros2 control set_state ... and added an optional parameter to ros2 control load ... to support the same logic, advancing the state of the controller to a desired state with supplying --state ... to the load command. With this, the previous "multitool" CLI verbs are considered deprecated configure_start_controller, load_configure_controller and load_start_controller.
We currently don't have documentation or functional tests for our CLI verbs.
For tests I don't think we need more than simplistic integration tests that verify that they exist and work as expected, regardless of back-end changes.
For documentation, I will add an initial version to the package and extend the control.ros.org scripts to pick it up and compile it into the page as well.
Opinions? @destogl @Karsten1987 @AndyZe ?
With #349 we've introduced a new verb,
ros2 control set_state ...and added an optional parameter toros2 control load ...to support the same logic, advancing the state of the controller to a desired state with supplying--state ...to the load command. With this, the previous "multitool" CLI verbs are considered deprecatedconfigure_start_controller,load_configure_controllerandload_start_controller.We currently don't have documentation or functional tests for our CLI verbs.
For tests I don't think we need more than simplistic integration tests that verify that they exist and work as expected, regardless of back-end changes.
For documentation, I will add an initial version to the package and extend the control.ros.org scripts to pick it up and compile it into the page as well.
Opinions? @destogl @Karsten1987 @AndyZe ?