BehaviorTree / BehaviorTree.ROS2

BehaviorTree.CPP utilities to work with ROS2
Apache License 2.0
144 stars 59 forks source link

Constant timeout in RosServiceNode #45

Open fmros opened 7 months ago

fmros commented 7 months ago

I stumbled over a use-case where a service call is taking longer than 1 second. I do not want to globally increase the service timeout over the BT::RosNodeParams parameter as this would affect all plugins I am using.

Is there any concern about removing the const declaration of servicetimeout variable for the ROS2 service timeout here? This way one could overwrite the value in the setRequest() function (and get the timeout value eventually over an input port).

Let me know and i will file a PR.