IntelRealSense / librealsense

Intel® RealSense™ SDK
https://www.intelrealsense.com/
Apache License 2.0
7.56k stars 4.82k forks source link

Rs-Converter filename timestamps dont match ros message timestamps #8678

Open geoeo opened 3 years ago

geoeo commented 3 years ago

Required Info
Camera Model { R200 / F200 / SR300 / ZR300 / D400 }
Firmware Version 2.40
Operating System & Version {Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS
Kernel Version (Linux Only) (e.g. 4.14.13)
Platform PC/Raspberry Pi/ NVIDIA Jetson / etc..
SDK Version { legacy / 2.<?>.<?> }
Language {C/C#/labview/nodejs/opencv/pcl/python/unity }
Segment {Robot/Smartphone/VR/AR/others }

Issue Description

This is a issue with rs-converter in tools, not with the specific realsense api. The timestamps in the image conversion are a factor 1000 greater than the ros message timestamps, this makes it confusing for user to write their own conversion, especially since imu conversion is not done via the rs-converter tool.

realsense_rosbag_capture converted_images

Either the timestamps of the filenames match the ros message timestamps or the user is informed that this is the case in the Readme

MartyG-RealSense commented 3 years ago

Hi @geoeo I have applied the Documentation tag to this case so that your suggested change to the readme of rs-convert considered by the RealSense team at a future date. This case should be kept open whilst that documentation change request is active.

In regard to support for IMU data in rs-convert, an official feature request has already been filed to consider adding this feature to the tool.

https://github.com/IntelRealSense/librealsense/issues/8320