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

Driver should support IR-RGB and RGB colorization #13

Open
shaun-edwards opened this issue Jan 20, 2019 · 1 comment
Open

Driver should support IR-RGB and RGB colorization #13

shaun-edwards opened this issue Jan 20, 2019 · 1 comment

Comments

@shaun-edwards
Copy link

A recent change was made to support IR-RGB publishing/pointcloud colorization (#12). This change inadvertently broke other usage (#12 (comment)).

The change was introduced to address synchronization issues between the realsense RGB camera and the pointcloud. In the prior version, the pointcloud and RGB were generated by different cameras. Synchronization was performed by timestamp syncing, which sometimes meant frames were mismatched by as much as 1/2 the frame rate. This is problematic when dealing with moving captures.
Because the IR-RGB image is directly used in generating the pointcloud, the two are exactly synchronized. An additional side benefit is that the image and pointcloud are represented in the same frame, making pixel to pixel mapping very efficient. The only downside, is that the IR-RGB image has some small IR artifacts that cause the image to look slightly different from the RGB image. Mainly, the IR-RGB image is less sensitive to red color.

I recommend that all applications switch to the IR-RGB image from the RGB image for the improvements stated above. However, to support older usage, we should consider modifying the driver to support both. Allowing both RGB and IR-RGB image streams and some kind of switch to control which is used for colorized pointcloud mapping.

@130s
Copy link

130s commented Jan 21, 2019

ir-rgb branch is created to keep the work for this WIP.

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

No branches or pull requests

2 participants