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

Update README.md #54

Open
wants to merge 1 commit into
base: kinetic-devel
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,10 @@ Robotic product handling simulation

## Getting Started

The project is based on Ubuntu 16.04, ROS Kinect, gazebo 7.0, wstool and catkin tools. Links to the instructions for installing ROS, Gazebo, etc. are provided below
The project is based on Ubuntu 16.04, ROS Kinetic, gazebo 7.0, wstool and catkin tools. Links to the instructions for installing ROS, Gazebo, etc. are provided below
- [Install ROS](http://wiki.ros.org/kinetic/Installation/Ubuntu)
- [Install gazebo 7.0](http://gazebosim.org/tutorials?tut=install_ubuntu&cat=install)
-Note: This may already be installed with ROS, do not overwrite gazebo 7.0 if it is already on your system
- [Install wstool](http://wiki.ros.org/wstool#Installation)
- [Install catkin-tools](http://catkin-tools.readthedocs.io/en/latest/installing.html)

Expand Down Expand Up @@ -43,10 +44,10 @@ The project is based on Ubuntu 16.04, ROS Kinect, gazebo 7.0, wstool and catkin
## Visualize The Simulation Enviroment

### Setup
- Source the catkin workspace
- Source the gilbreth workspace
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps we should say "gilbreth catkin" workspace


```
cd ~/catkin_ws
cd ~/gilbreth_ws
source devel/setup.bash
```

Expand All @@ -68,7 +69,7 @@ The project is based on Ubuntu 16.04, ROS Kinect, gazebo 7.0, wstool and catkin
```
roscd gilbreth_gazebo
source scripts/env_setup.bash
Copy link
Contributor

Choose a reason for hiding this comment

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

I think running this script is no longer necessary, can you test that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have tested this and it seems that you are correct, this script is unnecessary to run the gazebo environment.

roslaunch gilbreth_gazebo gilbreth.launch rviz:=false
roslaunch gilbreth_gazebo gilbreth_environment.launch rviz:=false
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this has been fixed in another one of your PR's #57

```

- The **source scripts/env_setup.bash** command sets up environment variables needed
Expand Down