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

rs2::syncer not working on Windows 11 / 2.50 #10258

Closed
roelofvandijk opened this issue Feb 22, 2022 · 10 comments
Closed

rs2::syncer not working on Windows 11 / 2.50 #10258

roelofvandijk opened this issue Feb 22, 2022 · 10 comments

Comments

@roelofvandijk
Copy link

roelofvandijk commented Feb 22, 2022


Required Info
Camera Model D400
Firmware Version 5.13.00.50
Operating System & Version Win 11
Kernel Version (Linux Only)
Platform PC
SDK Version 2.50
Language C++
Segment -

Issue Description

After upgrading from 2.38 on Windows 10 to 2.50 on Windows 11, using a rs2::syncer to sync frames from 2 D415 cameras does not work anymore.

This comment contains a minimal example that seems to be broken now: #4158 (comment)

The syncer now returns a frameset only containing half of the expected frames.

This issue points towards a potential change that causes this: #9030

Is there any way to make this work using the current release, or should I revert to an older one?

The situation before: rs2::syncer yielded one frameset with 6 frames
The current situation: rs2::syncer yields framesets with 3 frames, one per device, interleaved.

@roelofvandijk
Copy link
Author

roelofvandijk commented Feb 23, 2022

I have confirmed that the hardware timestamps are working on Windows 11, so that can't be it.

@roelofvandijk roelofvandijk changed the title Multi-device sync via rs2::syncer broken rs2::syncer not working on Windows 11 / 2.50 Feb 23, 2022
@MartyG-RealSense
Copy link
Collaborator

Hi @roelofvandijk If you suspect that the problem that you are experiencing began with SDK 2.43.0 then a logical starting point in investigating this case would be to install the 2.42.0 version of the full RealSense SDK for Windows to see whether your program works currently under that SDK version.

https://github.com/IntelRealSense/librealsense/releases/tag/v2.42.0

image

The syncer changes that were introduced in 2,43.0 are quoted below in an extract from the 2.43.0 release notes.

image

@MartyG-RealSense
Copy link
Collaborator

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

@roelofvandijk
Copy link
Author

Hi @MartyG-RealSense,

The current syncer behaviour seems like a regression that makes a common use-case (software-sync for frames from separate cameras) impossible.

The syncer currently alternates between framesets of a single D415 unit, while the previous (and desired) behaviour was receiving single synchronized framesets that contained all frames from both D415 units.

It would be great if Intel could reproduce this regression and - if confirmed - solve this in a next release.

If I am doing something wrong, it would be great to know how to achieve software-based sync for two different D415 devices using the current version of the SDK (2.50).

@MartyG-RealSense
Copy link
Collaborator

If you only need to capture a single frame from all attached D415 cameras simultaneously and export the frames as a PNG image then there is a C++ script shared by a RealSense user at #2219 (comment) that can do this and does not make use of syncer.

Conceivably if you needed to capture multiple frames then you could just loop the script round continuously until it is stopped.

@MartyG-RealSense
Copy link
Collaborator

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

@roelofvandijk
Copy link
Author

Hello @MartyG-RealSense, it would be great if this is either confirmed as a regression with a forthcoming fix, or if you can confirm that this is the intended behavior for the syncer now (no support for multiple devices anymore).

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Mar 7, 2022

A number of changes have been made to syncer inbetween SDKs 2.48.0 and 2.50.0.

2.48.0

image

2.49.0

image

2.50

image

image


It is unlikely that multiple device support in syncer would deliberately be removed. If other RealSense users can identify a potential bug when using syncer with multiple cameras and resolve it, I would encourage them to submit the fix as a Pull Request so that it can be considered for merging into the SDK.

https://github.com/IntelRealSense/librealsense/pulls

@MartyG-RealSense
Copy link
Collaborator

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

@MartyG-RealSense
Copy link
Collaborator

Case closed due to no further comments received.

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

No branches or pull requests

2 participants