ouster-lidar · Samahu · Jun 23, 2023 · May 26, 2023 · May 31, 2023 · May 31, 2023
diff --git a/.github/workflows/docker-image.yml b/.github/workflows/docker-image.yml
@@ -17,6 +17,7 @@ jobs:
         ros_distro:
           - rolling
           - humble
+          - iron
     steps:
       - uses: actions/checkout@v3
         with:

diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -27,6 +27,20 @@ ouster_ros(2)
 * bugfix: fixed an issue that prevents running multiple instances of the sensor and cloud components
   in the same process.
 * switch to using static transform publisher for the ros2 driver.
+* implemented a new node named ``os_driver`` which combines the functionality of ``os_sensor``,
+  ``os_cloud`` and ``os_image`` into a single node.
+* added support to parse the same parameters provided by the ``ros2_ouster_driver``, the parameters
+  are ``lidar_ip``, ``computer_ip``, ``proc_mask`` and ``use_system_default_qos``; the parameters
+  are fully functional and similar to what the ``ros2_ouster_driver`` provides.
+* for convenience introduced a new launch file ``driver_launch.py`` that is compatible with the 
+  ``ros2_ouster_driver`` in terms of parameters it accepts and the name of published topics.
+* introduced a new parameter ``point_cloud_frame`` to allow users to select which frame to use when
+  publishing the point cloud (choose between ``sensor`` and ``lidar``).
+* breaking: ``lidar`` frame is the default frame used when publishing point clouds.
+* added the ability to choose between ``SensorDataQoS`` or ``SystemDefaultQoS`` across all published
+  topics with ``SensorDataQoS`` selected by default for live sensor mode and ``SystemDefaultQoS``
+  enabled for record and replay modes.
+* introduced a new topic ``/ouster/scan`` which publishes ``sensor_msgs::msg::LaserScan`` messages
 
 ouster_client
 -------------

diff --git a/Dockerfile b/Dockerfile
@@ -53,10 +53,14 @@ RUN set -xe \
 FROM build-env
 
 SHELL ["/bin/bash", "-c"]
+
 RUN source /opt/ros/$ROS_DISTRO/setup.bash && colcon build \
     --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release \
     -DCMAKE_CXX_FLAGS="-Wno-deprecated-declarations"
 
+RUN source /opt/ros/$ROS_DISTRO/setup.bash && colcon test \
+    --ctest-args tests ouster_ros --rerun-failed --output-on-failure
+
 # Entrypoint for running Ouster ros:
 #
 # Usage: docker run --rm -it ouster-ros [sensor.launch parameters ..]

diff --git a/README.md b/README.md
@@ -1,7 +1,7 @@
 # Official ROS driver for Ouster sensors
 
 [ROS1 (melodic/noetic)](https://github.com/ouster-lidar/ouster-ros/tree/master) |
-[ROS2 (rolling/humble)](https://github.com/ouster-lidar/ouster-ros/tree/ros2) |
+[ROS2 (rolling/humble/iron)](https://github.com/ouster-lidar/ouster-ros/tree/ros2) |
 [ROS2 (foxy)](https://github.com/ouster-lidar/ouster-ros/tree/ros2-foxy)
 
 <p style="float: right;"><img width="20%" src="docs/images/logo.png" /></p>
@@ -12,20 +12,25 @@
 | ROS2 (rolling/humble) | [![rolling/humble](https://github.com/ouster-lidar/ouster-ros/actions/workflows/docker-image.yml/badge.svg?branch=ros2)](https://github.com/ouster-lidar/ouster-ros/actions/workflows/docker-image.yml)
 | ROS2 (foxy) | [![foxy](https://github.com/ouster-lidar/ouster-ros/actions/workflows/docker-image.yml/badge.svg?branch=ros2-foxy)](https://github.com/ouster-lidar/ouster-ros/actions/workflows/docker-image.yml)
 
-- [Overview](#overview)
-- [Requirements](#requirements)
-- [Getting Started](#getting-started)
-- [Usage](#usage)
-  - [Launching Nodes](#launching-nodes)
-    - [Sensor Mode](#sensor-mode)
-    - [Recording Mode](#recording-mode)
-    - [Replay Mode](#replay-mode)
-    - [Multicast Mode (experimental)](#multicast-mode-experimental)
-  - [Invoking Services](#invoking-services)
-    - [GetMetadata](#getmetadata)
-    - [GetConfig](#getconfig)
-    - [SetConfig (experimental)](#setconfig-experimental)
-- [License](#license)
+- [Official ROS driver for Ouster sensors](#official-ros-driver-for-ouster-sensors)
+  - [Overview](#overview)
+  - [Requirements](#requirements)
+    - [Linux](#linux)
+    - [Windows](#windows)
+    - [Mac](#mac)
+  - [Getting Started](#getting-started)
+  - [Usage](#usage)
+    - [Launching Nodes](#launching-nodes)
+      - [Sensor Mode](#sensor-mode)
+      - [Recording Mode](#recording-mode)
+      - [Replay Mode](#replay-mode)
+      - [Multicast Mode (experimental)](#multicast-mode-experimental)
+    - [Invoking Services](#invoking-services)
+      - [GetMetadata](#getmetadata)
+      - [GetConfig](#getconfig)
+      - [SetConfig](#setconfig)
+      - [Reset](#reset)
+  - [License](#license)
 
 
 ## Overview
@@ -120,13 +125,21 @@ source ros2_ws/install/setup.bash
 
 ### Launching Nodes
 The package supports three modes of interaction, you can connect to a _live sensor_, _replay_ a recorded bag or _record_ a new bag file using the corresponding launch files. Recently, we have
-added a new mode that supports multicast. The commands are listed below:
+added a new mode that supports multicast. The commands are listed below, for convenience we do
+provide both launch file formats (xml and python) but the python format is recommended:
 
 #### Sensor Mode
+To connect to a live sensor you use the following launch file
 ```bash
 ros2 launch ouster_ros sensor.launch.xml    \
     sensor_hostname:=<sensor host name>
 ```
+The equivalent python file is 
+```bash
+ros2 launch ouster_ros driver.launch.py    \
+    params_files:=<path to params yaml file>
+```
+If you don't pass a params_file then the file located at `ouster/config/driver_params.yaml` will be used
 
 #### Recording Mode
 > Note
@@ -197,14 +210,14 @@ To get the current config of a live sensor, invoke the command:
 ros2 service call /ouster/get_config ouster_srvs/srv/GetConfig
 ```
 
-#### SetConfig (experimental)
+#### SetConfig
 To change config via a file while connected to a live sensor, invoke the command:
 ```bash
 ros2 service call /ouster/set_config ouster_srvs/srv/SetConfig \
     "{config_file: 'some_config.json'}"
 ```
 
-#### Reset (experimental)
+#### Reset
 To reset the new reset service, execute the following commnad:
 ```bash
 ros2 service call /ouster/reset std_srvs/srv/Empty

diff --git a/ouster-ros/CMakeLists.txt b/ouster-ros/CMakeLists.txt
@@ -156,6 +156,28 @@ rclcpp_components_register_node(os_image_component
   EXECUTABLE os_image
 )
 
+# ==== os_sensor_component ====
+create_ros2_component(os_driver_component
+  "src/os_sensor_node_base.cpp;src/os_sensor_node.cpp;src/os_driver_node.cpp"
+  "std_srvs"
+)
+rclcpp_components_register_node(os_driver_component
+  PLUGIN "ouster_ros::OusterDriver"
+  EXECUTABLE os_driver
+)
+
+
+# ==== Test ====
+if(BUILD_TESTING)
+  find_package(ament_cmake_gtest REQUIRED)
+  ament_add_gtest(${PROJECT_NAME}_test test/ring_buffer_test.cpp)
+  target_include_directories(${PROJECT_NAME}_test PUBLIC
+    $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
+    $<INSTALL_INTERFACE:include>
+  )
+endif()
+
+
 # ==== Install ====
 install(
   TARGETS
@@ -164,6 +186,7 @@ install(
     os_replay_component
     os_cloud_component
     os_image_component
+    os_driver_component
   ARCHIVE DESTINATION lib
   LIBRARY DESTINATION lib
   RUNTIME DESTINATION bin
-Original file line number
+Diff line change
@@ Expand Up / @@ -17,6 +17,7 @@ jobs: @@
             ros_distro:
               - rolling
               - humble
+              - iron
         steps:
           - uses: actions/checkout@v3
             with:
@@ Expand Down @@