Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Port wheel slip plugin to ros2 (forward port from eloquent)#1148

Merged
scpeters merged 3 commits intoros-simulation:ros2from
scpeters:wheel_slip_plugin_ros2
Aug 17, 2020
Merged

Port wheel slip plugin to ros2 (forward port from eloquent)#1148
scpeters merged 3 commits intoros-simulation:ros2from
scpeters:wheel_slip_plugin_ros2

Conversation

@scpeters
Copy link
Copy Markdown
Member

This is a forward port of #1099.

Initial port of gazebo_ros_wheel_slip_plugin to ros2.

It includes a similar change to #1111, which moved the
WheelSlipPlugin::Load call before the parameter callback
so that the values from SDF/URDF are set first.
Then the callback is changed to ignore negative values, and the
default slip parameter values are set to -1, so that the SDF/URDF
values are still preferred unless a different parameter value
is specified in a launch file.

Initial port of gazebo_ros_wheel_slip_plugin to ros2.

It includes a similar change to ros-simulation#1111, which moved the
WheelSlipPlugin::Load call before the parameter callback
so that the values from SDF/URDF are set first.
Then the callback is changed to ignore negative values, and the
default slip parameter values are set to -1, so that the SDF/URDF
values are still preferred unless a different parameter value
is specified in a launch file.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@scpeters scpeters requested a review from jacobperron August 13, 2020 05:47
Comment thread gazebo_plugins/CMakeLists.txt
Co-authored-by: Jacob Perron <jacob@openrobotics.org>
Copy link
Copy Markdown
Collaborator

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (pending CI)

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@scpeters scpeters merged commit bc4938f into ros-simulation:ros2 Aug 17, 2020
@scpeters scpeters deleted the wheel_slip_plugin_ros2 branch August 17, 2020 18:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants