Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added initial version of gazebo_ros2_control #1

Merged
merged 8 commits into from
Jun 1, 2020

Conversation

ahcorde
Copy link
Collaborator

@ahcorde ahcorde commented May 28, 2020

Based on the effort done by @ddengster in this PR ros-simulation/gazebo_ros_pkgs#1083 porting gazebo_ros_control to ROS 2. I have been working in reproduce the results. I tried "fix" some of the issue that I found in the code and comments

@chapulina @Karsten1987

I know this is not a final solution. It's just make it work in ROS 2.

Signed-off-by: ahcorde ahcorde@gmail.com

Signed-off-by: ahcorde <ahcorde@gmail.com>
Copy link
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

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

Thanks for reviving this.
Are you planning to add a CI as well?

gazebo_ros2_control/CHANGELOG.rst Outdated Show resolved Hide resolved
gazebo_ros2_control/README.md Outdated Show resolved Hide resolved
gazebo_ros2_control/src/default_robot_hw_sim.cpp Outdated Show resolved Hide resolved
gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp Outdated Show resolved Hide resolved
gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp Outdated Show resolved Hide resolved
gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp Outdated Show resolved Hide resolved
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Copy link
Collaborator

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

I haven't tried it out, but I just left some general comments about style, linting, etc.

gazebo_ros2_control/README.md Outdated Show resolved Hide resolved
gazebo_ros2_control/CMakeLists.txt Show resolved Hide resolved
gazebo_ros2_control/package.xml Outdated Show resolved Hide resolved
gazebo_ros2_control/package.xml Outdated Show resolved Hide resolved
gazebo_ros2_control/src/default_robot_hw_sim.cpp Outdated Show resolved Hide resolved
gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp Outdated Show resolved Hide resolved
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Copy link
Collaborator

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

Thanks for addressing all my nitpicks! I just took another high-level look and it looks like a great start!

@ahcorde
Copy link
Collaborator Author

ahcorde commented Jun 1, 2020

Great! thank you for the reviews!

@ahcorde ahcorde merged commit 98834ef into master Jun 1, 2020
@ahcorde ahcorde deleted the ahcorde/initial/gazebo_ros2_control branch June 1, 2020 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants