Skip to content

Commit efbdeab

Browse files
authored
Increase default dashboard client timeout to 2 seconds (#1564)
The default timeout of 1 second can collide with certain actions. For example, the power_off command on PolyScope 5 takes 1 sec almost exactly until it sends the answer. With that, the timeout was right on the edge and led to a flaky integration test. Defaulting to 2 seconds should prevent those tests from failing randomly.
1 parent f247c0b commit efbdeab

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ur_robot_driver/src/dashboard_client_ros.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ namespace ur_robot_driver
5151
DashboardClientROS::DashboardClientROS(const rclcpp::Node::SharedPtr& node, const std::string& robot_ip)
5252
: node_(node), primary_client_(robot_ip, notifier_)
5353
{
54-
node_->declare_parameter<double>("receive_timeout", 1);
54+
node_->declare_parameter<double>("receive_timeout", 2);
5555

5656
primary_client_.start(10, std::chrono::seconds(10));
5757
auto robot_version = primary_client_.getRobotVersion();

0 commit comments

Comments
 (0)