-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Stabilize timestamp domain test #11167
Stabilize timestamp domain test #11167
Conversation
two queues were created for each type of sensor. A queue was increased to 10 frames.
I see the capacity is 1, please update |
It was 1 in the previous code version, but I checked with size=10 yesterday and found it failing the test. |
The capacity of a queue is 5. In case of a failure, data fame printed.
No need for more then 1 |
Unnecessary "IF" statement was removed. The frame queue size changed back to 1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Tracked on [LRS-567]
two queues were created for each type of sensor.
A queue was increased to 1 frame.