magazino / pylon_camera

ROS-Driver for Basler Cameras
BSD 3-Clause "New" or "Revised" License
78 stars 108 forks source link

Installation instructions unclear #21

Closed themightyoarfish closed 6 years ago

themightyoarfish commented 7 years ago

The instructions say

The package has been tested for ROS-Indigo. For other ROS versions, please replace 'indigo' with your desired ROS-Version in the following install instructions.

However, the only part where indigo occurs is in sudo sh -c 'echo "yaml https://raw.githubusercontent.com/magazino/pylon_camera/indigo-devel/rosdep/pylon_sdk.yaml " > /etc/ros/rosdep/sources.list.d/15-plyon_camera.list'

Replacing indigo with something else will error since the file won't exist. So it's not clear if and how the installation instructions must be adapted for a different distro.

What's also not explicitly mentioned that that the Basler pylon5 camera driver must be installed before using these instructions. With that, installation on indigo seems to work without problems.

andrejpan commented 7 years ago

Strongly supporting to update the instructions.