Remove camera node

The existing camera node doesn't publish a camera_info topic, so it's not compatible with the previous commits. Also update the README to suggest existing ROS drivers as alternatives.
kini5gowda · Jun 13, 2019 · 6c5040d · 6c5040d
1 parent 110f214
commit 6c5040d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 58 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -166,7 +166,6 @@ include_directories(
 # )
 
 catkin_install_python(PROGRAMS
-  nodes/camera
   nodes/dope
   scripts/train.py
   DESTINATION ${CATKIN_PACKAGE_BIN_DESTINATION})

diff --git a/nodes/camera b/nodes/camera
diff --git a/readme.md b/readme.md
@@ -58,12 +58,15 @@ This is the official DOPE ROS package for detection and 6-DoF pose estimation of
       $ roscore
       ```
 
-2. **Start camera node** (or start your own camera node)
-      ```
-      $ rosrun dope camera  # Publishes RGB images to `/dope/webcam_rgb_raw`
-      ```
-  
+2. **Start camera node**
 
+    Start the ROS driver for your camera. If you are using a webcam, try one of these:
+
+    * [usb_cam](http://wiki.ros.org/usb_cam)
+    * [libuvc_camera](http://wiki.ros.org/libuvc_camera)
+
+    The camera must publish a correct `camera_info` topic to enable DOPE to compute the correct poses. Basically all ROS drivers have a `camera_info_url` parameter where you can set the calibration info (but most ROS drivers include a reasonable default).
+  
 3. **Edit config info** (if desired) in `~/catkin_ws/src/dope/config/config_pose.yaml`
     * `topic_camera`: RGB topic to listen to
     * `topic_camera_info`: camera info topic to listen to