Action Server to execute trees #60
Conversation
There was a problem hiding this comment.
What do you think about moving this file to the behaviortree_ros2 package and renaming it to bt_server or something along those lines? For the most part people should be able to use this node to run the library and it seems weird to have launch files calling <node pkg="btcpp_ros2_samples" exec="sample_bt_executor"> when it's a very capable node and not just a sample.
|
I added some basic docs and cleaned up a few things so the examples run, let me know what you think. I was traveling most of today but will try and look into the other changes you made later. |
| RCLCPP_INFO(kLogger, "Loaded ROS Plugin: %s", filename.c_str()); | ||
| } | ||
| catch(const std::exception& e) | ||
| { | ||
| RCLCPP_ERROR(kLogger, "Failed to load ROS Plugin: %s \n %s", filename.c_str(), | ||
| e.what()); |
There was a problem hiding this comment.
We can drop ROS from these prints now that it does both types.
| TreeExecutionServer::~TreeExecutionServer() | ||
| {} |
There was a problem hiding this comment.
Do we need to handle joining and cleaning up the action_thread here?
|
I have a fix for pre-commit and minor changes to the docs if you are okay with me pushing it. |
|
I was going to merge it... right now! Waiting for CI to turn green. Thanks! |
|
I will wait for your changes to the docs to be pushed |
There was a problem hiding this comment.
Thanks for all the help here @facontidavide. My changes were to fix the file permissions (which you fixed) and some minor white space but this looks good as is.
Thanks to the hard work of @MarqRazz , we have here a first draft of a "standard" action server to execute trees on demand.
Virtual methods add a number of customization points, that are hopefully general enough to cover 99% of the cases.