Skip to content

Commit e16f38e

Browse files
committed
Apply suggestions from code review
1 parent e9f74ee commit e16f38e

File tree

3 files changed

+5
-6
lines changed

3 files changed

+5
-6
lines changed

ros2_control_demo_hardware/ros2_control_demo_hardware.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
type="ros2_control_demo_hardware::DiffBotSystemHardware"
1818
base_class_type="hardware_interface::SystemInterface">
1919
<description>
20-
The ROS2 Control minimal robot protocol. This is example for start porting the robot from ROS 1.
20+
The ros2_control DiffBot example using a system hardware interface-type. It uses velocity command and position state interface. The example is the starting point to implement a hardware interface for differential-drive mobile robots.
2121
</description>
2222
</class>
2323
</library>

ros2_control_demo_hardware/src/diffbot_system.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright 2020 ros2_control Development Team
1+
// Copyright 2021 ros2_control Development Team
22
//
33
// Licensed under the Apache License, Version 2.0 (the "License");
44
// you may not use this file except in compliance with the License.
@@ -118,7 +118,6 @@ DiffBotSystemHardware::export_command_interfaces()
118118
return command_interfaces;
119119
}
120120

121-
122121
return_type DiffBotSystemHardware::start()
123122
{
124123
RCLCPP_INFO(
@@ -215,7 +214,6 @@ hardware_interface::return_type ros2_control_demo_hardware::DiffBotSystemHardwar
215214
} // namespace ros2_control_demo_hardware
216215

217216
#include "pluginlib/class_list_macros.hpp"
218-
219217
PLUGINLIB_EXPORT_CLASS(
220218
ros2_control_demo_hardware::DiffBotSystemHardware,
221219
hardware_interface::SystemInterface

ros2_control_demo_robot/controllers/diffbot_diff_drive_controller.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ controller_manager:
44

55
joint_state_controller:
66
type: joint_state_controller/JointStateController
7+
78
diffbot_base_controller:
89
type: diff_drive_controller/DiffDriveController
910

1011
diffbot_base_controller:
1112
ros__parameters:
1213
left_wheel_names: ["left_wheel_joint"]
1314
right_wheel_names: ["right_wheel_joint"]
14-
#write_op_modes: ["motor_controller"]
1515

1616
wheel_separation: 0.10
1717
#wheels_per_side: 1 # actually 2, but both are controlled by 1 signal
@@ -63,4 +63,5 @@ diffbot_base_controller:
6363
angular.z.max_acceleration: 1.0
6464
angular.z.min_acceleration: -1.0
6565
angular.z.max_jerk: 0.0
66-
angular.z.min_jerk: 0.0
66+
angular.z.min_jerk: 0.0
67+

0 commit comments

Comments
 (0)