Skip to content

Commit 5b87cf3

Browse files
authored
Merge pull request #118 from iche033/iche033/fix_humble_demos
Fix example demos in humble branch
2 parents cd72a4f + 8b2b2fc commit 5b87cf3

File tree

5 files changed

+29
-23
lines changed

5 files changed

+29
-23
lines changed

ign_ros2_control/src/ign_ros2_control_plugin.cpp

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -355,10 +355,10 @@ void IgnitionROS2ControlPlugin::Configure(
355355
// setup actuators and mechanism control node.
356356
// This call will block if ROS is not properly initialized.
357357
std::string urdf_string;
358-
std::vector<hardware_interface::HardwareInfo> control_hardware;
358+
std::vector<hardware_interface::HardwareInfo> control_hardware_info;
359359
try {
360360
urdf_string = this->dataPtr->getURDF();
361-
control_hardware = hardware_interface::parse_control_resources_from_urdf(urdf_string);
361+
control_hardware_info = hardware_interface::parse_control_resources_from_urdf(urdf_string);
362362
} catch (const std::runtime_error & ex) {
363363
RCLCPP_ERROR_STREAM(
364364
this->dataPtr->node_->get_logger(),
@@ -381,15 +381,15 @@ void IgnitionROS2ControlPlugin::Configure(
381381
return;
382382
}
383383

384-
for (unsigned int i = 0; i < control_hardware.size(); ++i) {
385-
std::string robot_hw_sim_type_str_ = control_hardware[i].hardware_class_type;
384+
for (unsigned int i = 0; i < control_hardware_info.size(); ++i) {
385+
std::string robot_hw_sim_type_str_ = control_hardware_info[i].hardware_class_type;
386386
auto ignitionSystem = std::unique_ptr<ign_ros2_control::IgnitionSystemInterface>(
387387
this->dataPtr->robot_hw_sim_loader_->createUnmanagedInstance(robot_hw_sim_type_str_));
388388

389389
if (!ignitionSystem->initSim(
390390
this->dataPtr->node_,
391391
enabledJoints,
392-
control_hardware[i],
392+
control_hardware_info[i],
393393
_ecm,
394394
this->dataPtr->update_rate))
395395
{
@@ -398,8 +398,14 @@ void IgnitionROS2ControlPlugin::Configure(
398398
return;
399399
}
400400

401-
resource_manager_->import_component(std::move(ignitionSystem), control_hardware[i]);
401+
resource_manager_->import_component(std::move(ignitionSystem), control_hardware_info[i]);
402+
403+
rclcpp_lifecycle::State state(
404+
lifecycle_msgs::msg::State::PRIMARY_STATE_ACTIVE,
405+
hardware_interface::lifecycle_state_names::ACTIVE);
406+
resource_manager_->set_component_state(control_hardware_info[i].name, state);
402407
}
408+
403409
// Create the controller manager
404410
RCLCPP_INFO(this->dataPtr->node_->get_logger(), "Loading controller_manager");
405411
this->dataPtr->controller_manager_.reset(

ign_ros2_control_demos/launch/cart_example_effort.launch.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def generate_launch_description():
5252
)
5353

5454
ignition_spawn_entity = Node(
55-
package='ros_ign_gazebo',
55+
package='ros_gz_sim',
5656
executable='create',
5757
output='screen',
5858
arguments=['-string', doc.toxml(),
@@ -61,13 +61,13 @@ def generate_launch_description():
6161
)
6262

6363
load_joint_state_broadcaster = ExecuteProcess(
64-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
64+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
6565
'joint_state_broadcaster'],
6666
output='screen'
6767
)
6868

6969
load_joint_trajectory_controller = ExecuteProcess(
70-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start', 'effort_controllers'],
70+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active', 'effort_controllers'],
7171
output='screen'
7272
)
7373

@@ -77,7 +77,7 @@ def generate_launch_description():
7777
PythonLaunchDescriptionSource(
7878
[os.path.join(get_package_share_directory('ros_ign_gazebo'),
7979
'launch', 'ign_gazebo.launch.py')]),
80-
launch_arguments=[('ign_args', [' -r -v 3 empty.sdf'])]),
80+
launch_arguments=[('gz_args', [' -r -v 3 empty.sdf'])]),
8181
RegisterEventHandler(
8282
event_handler=OnProcessExit(
8383
target_action=ignition_spawn_entity,

ign_ros2_control_demos/launch/cart_example_position.launch.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def generate_launch_description():
5252
)
5353

5454
ignition_spawn_entity = Node(
55-
package='ros_ign_gazebo',
55+
package='ros_gz_sim',
5656
executable='create',
5757
output='screen',
5858
arguments=['-string', doc.toxml(),
@@ -61,13 +61,13 @@ def generate_launch_description():
6161
)
6262

6363
load_joint_state_broadcaster = ExecuteProcess(
64-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
64+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
6565
'joint_state_broadcaster'],
6666
output='screen'
6767
)
6868

6969
load_joint_trajectory_controller = ExecuteProcess(
70-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
70+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
7171
'joint_trajectory_controller'],
7272
output='screen'
7373
)
@@ -78,7 +78,7 @@ def generate_launch_description():
7878
PythonLaunchDescriptionSource(
7979
[os.path.join(get_package_share_directory('ros_ign_gazebo'),
8080
'launch', 'ign_gazebo.launch.py')]),
81-
launch_arguments=[('ign_args', [' -r -v 4 empty.sdf'])]),
81+
launch_arguments=[('gz_args', [' -r -v 4 empty.sdf'])]),
8282
RegisterEventHandler(
8383
event_handler=OnProcessExit(
8484
target_action=ignition_spawn_entity,

ign_ros2_control_demos/launch/cart_example_velocity.launch.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def generate_launch_description():
5252
)
5353

5454
ignition_spawn_entity = Node(
55-
package='ros_ign_gazebo',
55+
package='ros_gz_sim',
5656
executable='create',
5757
output='screen',
5858
arguments=['-string', doc.toxml(),
@@ -61,18 +61,18 @@ def generate_launch_description():
6161
)
6262

6363
load_joint_state_broadcaster = ExecuteProcess(
64-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
64+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
6565
'joint_state_broadcaster'],
6666
output='screen'
6767
)
6868

6969
load_joint_trajectory_controller = ExecuteProcess(
70-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start', 'velocity_controller'],
70+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active', 'velocity_controller'],
7171
output='screen'
7272
)
7373

7474
load_imu_sensor_broadcaster = ExecuteProcess(
75-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
75+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
7676
'imu_sensor_broadcaster'],
7777
output='screen'
7878
)
@@ -83,7 +83,7 @@ def generate_launch_description():
8383
PythonLaunchDescriptionSource(
8484
[os.path.join(get_package_share_directory('ros_ign_gazebo'),
8585
'launch', 'ign_gazebo.launch.py')]),
86-
launch_arguments=[('ign_args', [' -r -v 4 empty.sdf'])]),
86+
launch_arguments=[('gz_args', [' -r -v 4 empty.sdf'])]),
8787
RegisterEventHandler(
8888
event_handler=OnProcessExit(
8989
target_action=ignition_spawn_entity,

ign_ros2_control_demos/launch/diff_drive_example.launch.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def generate_launch_description():
5353
)
5454

5555
ignition_spawn_entity = Node(
56-
package='ros_ign_gazebo',
56+
package='ros_gz_sim',
5757
executable='create',
5858
output='screen',
5959
arguments=['-string', doc.toxml(),
@@ -62,13 +62,13 @@ def generate_launch_description():
6262
)
6363

6464
load_joint_state_controller = ExecuteProcess(
65-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
65+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
6666
'joint_state_broadcaster'],
6767
output='screen'
6868
)
6969

7070
load_joint_trajectory_controller = ExecuteProcess(
71-
cmd=['ros2', 'control', 'load_controller', '--set-state', 'start',
71+
cmd=['ros2', 'control', 'load_controller', '--set-state', 'active',
7272
'diff_drive_base_controller'],
7373
output='screen'
7474
)
@@ -79,7 +79,7 @@ def generate_launch_description():
7979
PythonLaunchDescriptionSource(
8080
[os.path.join(get_package_share_directory('ros_ign_gazebo'),
8181
'launch', 'ign_gazebo.launch.py')]),
82-
launch_arguments=[('ign_args', [' -r -v 4 empty.sdf'])]),
82+
launch_arguments=[('gz_args', [' -r -v 4 empty.sdf'])]),
8383
RegisterEventHandler(
8484
event_handler=OnProcessExit(
8585
target_action=ignition_spawn_entity,

0 commit comments

Comments
 (0)