Skip to content

Stretch Gazebo for ROS1 - #20

Merged
hello-binit merged 3 commits into
hello-robot:masterfrom
PickNikRobotics:pr-gazebo-tuning
Jun 16, 2021
Merged

Stretch Gazebo for ROS1#20
hello-binit merged 3 commits into
hello-robot:masterfrom
PickNikRobotics:pr-gazebo-tuning

Conversation

@vatanaksoytezer

@vatanaksoytezer vatanaksoytezer commented May 6, 2021

Copy link
Copy Markdown
Contributor

Ongoing PR to achieve a fully working Stretch on Gazebo

@vatanaksoytezer vatanaksoytezer changed the title WIP Gazebo ROS1 Tuned WIP: Gazebo ROS1 Tuned May 6, 2021
@vatanaksoytezer vatanaksoytezer changed the title WIP: Gazebo ROS1 Tuned WIP: Stretch Gazebo for ROS1 May 6, 2021
@vatanaksoytezer

vatanaksoytezer commented May 6, 2021

Copy link
Copy Markdown
Contributor Author

This should fully work now including all the sensors and actuators. I might want to tune the physics a little bit more, but as an initial work this should do the trick. I will squash this and will check the dependencies, then it will be ready to merge.

@vatanaksoytezer vatanaksoytezer changed the title WIP: Stretch Gazebo for ROS1 Stretch Gazebo for ROS1 Jun 7, 2021
@vatanaksoytezer

Copy link
Copy Markdown
Contributor Author

I documented, cleaned up, squashed and rebased this. @hello-binit this is ready for it's initial merge now. I would appreciate if I can get a review.

@hello-binit

Copy link
Copy Markdown
Contributor

Thanks @vatanaksoytezer, excited to test out the simulation. Currently running up against the following two issues when trying to build the package:

  1. Rosdep is unable to resolve realsense_gazebo_plugin
  2. catkin_make fails with the following error:
-- +++ processing catkin package: 'stretch_gazebo'
-- ==> add_subdirectory(stretch_ros/stretch_gazebo)
CMake Error at /opt/ros/melodic/share/catkin/cmake/custom_install.cmake:13 (_install):
  _install FILES given directory "config" to install.
Call Stack (most recent call first):
  stretch_ros/stretch_gazebo/CMakeLists.txt:8 (install)

Have you seen either of these error before?

@vatanaksoytezer

Copy link
Copy Markdown
Contributor Author

Hey @hello-binit I am Out of office today but here are quick ways you should be able to get it working:

  1. clone realsense gazebo plugin to your workspace
  2. delete config install from strecth gazebo.

I'll apply these fixes to this pr in the weekend

@hello-binit

Copy link
Copy Markdown
Contributor

Thanks! Looks good @vatanaksoytezer, I'll merge it in once you've had a chance to solve the realsense gazebo plugin and config install issues.

I'll also add a "Known Limitations" section to the Gazebo README later. For example, it seems the gazebo simulation has three different joint trajectory action controllers, which differs from the single controller stretch_core uses.

@vatanaksoytezer

vatanaksoytezer commented Jun 16, 2021

Copy link
Copy Markdown
Contributor Author

PickNikRobotics#45 here is the PR for ignition port and documentation @hello-binit

@hello-binit
hello-binit merged commit c7522ee into hello-robot:master Jun 16, 2021
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.

2 participants