-
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
Add a short stress test for start/stop #11664
Conversation
# License: Apache 2.0. See LICENSE file in root directory. | ||
# Copyright(c) 2023 Intel Corporation. All Rights Reserved. | ||
|
||
# test:device D400* |
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.
Why only D400?
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.
Other are about to be deprecated, no need to add run time for them
|
||
################################################################################################ | ||
test.start("Testing pipeline start/stop stress test") | ||
for i in range(10): |
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.
10 is not "stress" test. Can you add a test case that will run only on nightly
context with 1000 iterations?
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.
Its a mini stress test..
RealCI covers more.
The main idea is to see multiple start stop works.
1 min is enough
################################################################################################ | ||
test.start("Testing pipeline start/stop stress test") | ||
for i in range(10): | ||
log.out("starting iteration #", i, "/", 10) |
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.
Use i + 1
now last iteration is 9/10
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.
Done
import platform | ||
|
||
|
||
# Start depth + color streams and measure the time from stream opened until first frame arrived using pipeline API. |
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.
You also run IMU, better to write Start all streams...
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.
Changed
|
||
|
||
# Start depth + color streams and measure the time from stream opened until first frame arrived using pipeline API. | ||
# Verify that the time do not exceeds the maximum time allowed |
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.
does not
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.
Done
pipe = rs.pipeline() | ||
start_call_stopwatch = Stopwatch() | ||
pipe.start(config) | ||
# wait_for_frames will through if no frames received so no assert is needed |
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.
throw
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.
Done
Tracked on [LRS-751]
Test result