ouster-ros icon indicating copy to clipboard operation
ouster-ros copied to clipboard

Bring back optional publishing of lidar and imu packets to new single nodelet driver

Open skohlbr opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe. We recently upgraded to use the new single nodelet driver (started via driver.launch as described in https://github.com/ouster-lidar/ouster-ros#sensor-mode ). This shows significantly reduced CPU consumption on Jetson embedded systems compared to the "old" multi-nodelet setup. However, it appears the support to simultaneously publish the lidar and imu packets has been dropped. This is a problem because for data recording purposes, those are the most useful data to record (as also remarked by others in https://github.com/ouster-lidar/ouster-ros/issues/110 ).

Describe the solution you'd like It would be great to have the single driver nodelet also optionally (i.e. if numSubscribers() > 0 ) publish the raw packets, so they can be recorded for later postprocessing/analysis.

Describe alternatives you've considered An alternative is using the old multi-nodelet setup, but that one causes high CPU load on Jetson embedded systems, so not a good option.

Targeted Platform (please complete the following information only if applicable, otherwise dot N/A):

  • ROS1, but likely applies to ROS2 also (?)

skohlbr avatar Nov 08 '23 20:11 skohlbr

Same as #238

Samahu avatar Nov 08 '23 22:11 Samahu

@Samahu can you provide an indication if this is in the works or what the plans are wrt this feature?

skohlbr avatar Feb 20 '24 14:02 skohlbr