Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

single_depth_color_640x480.bag violates ROSBag file format specification #7845

Closed
MarekKnapek opened this issue Nov 23, 2020 · 4 comments
Closed
Labels

Comments

@MarekKnapek
Copy link

Hi, I'm in process of writing a ROSBag file format parser and I found single_depth_color_640x480.bag file in your repository, I'm using it as a unit test for my parser.

I would like to inform you, that this file violates ROSBag file format specification.

If you look at Bags/Format/2.0 - ROS Wiki page, paragraph Connection, you will see, that connection record must have specific body. In particular, the connection record body must contain a connection header. Connection header consists of mandatory and optional fields. Mandatory fields are topic, type, md5sum and message_definition.

Sadly, the file in your repository is missing the topic field and thus it is invalid ROSBag file.

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Nov 23, 2020

Hi @MarekKnapek The bag format used by the RealSense SDK is mostly based on the rosbag format but not every rosbag may be able to be played back in the SDK. Some have also found that an SDK-recorded bag will not play in the ROS bag playback. Advice about this is provided in the links below.

Rosbag playback in SDK
#3020 (comment)

SDK bag playback in ROS
#4260

In regard to a rosbag parser, there is an existing one listed in the Community Projects section of user-contributed projects at the bottom of the SDK Examples page.

#2215

The full list of Community Projects can be found here:

https://github.com/IntelRealSense/librealsense/tree/master/examples#community-projects

@MartyG-RealSense
Copy link
Collaborator

Hi @MarekKnapek Do you require further assistance with this case, please? Thanks!

@MarekKnapek
Copy link
Author

Hi @MartyG-RealSense, I don't need any assistance with this.

You should be sad, that file in your repository is not readable by standard ROSBag tools, but apparently, you are not.

Feel free to close this issue with something like won't-fix / as-designed / not-a-bug resolution tag.

Marek

@MartyG-RealSense
Copy link
Collaborator

I am sorry that you are disappointed in the design decision to focus on camera-only data. #3020 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants