ros-industrial / industrial_training

ROS-Industrial Training Material
https://industrial-training-master.readthedocs.io
Apache License 2.0
427 stars 238 forks source link

Training unit errors/updates/omissions #9

Open shaun-edwards opened 10 years ago

shaun-edwards commented 10 years ago
gavanderhoorn commented 10 years ago

in unit 4.1 (but possibly also in other units) the devel branches in the github repositories are refered to as groovy_devel (for Groovy). It should be groovy-devel (note the dash, instead of an underscore).

The steps in 4.1 also do not source either a workspace or the main distro setup.bash (but perhaps this is already explained in other units?).

shaun-edwards commented 10 years ago

We may need to rethink section 4.1. The use of catkin is new to the tutorials and doesn't really fit. I only added it because I wanted to demonstrate a git repo. Unfortunately we don't have a rosbuild git repo. FYI...the steps in 4.1 work because the "training_unit" command sources a setup file. These are all things we should consider when porting everything to catkin.

JeremyZoss commented 10 years ago

I fixed the MoveIt wiki/API links. The 3.6 training lesson seems to work fine for me. That named position target ("straight up") is defined in the moveit package created for that exercise. Perhaps you were using a different MoveIt config? Feel free to close this issue when you've verified that last issue.

airplanesrule commented 10 years ago

using the ros-industrial training disc image and running

training_unit demo_manipulation catkin_make

leads to dependency issues for me see: https://groups.google.com/forum/#!topic/swri-ros-pkg-dev/bNFCpZc5Xmw