Add a short stress test for start/stop - #11664
Merged
Merged
Conversation
OhadMeir
reviewed
Apr 10, 2023
| # License: Apache 2.0. See LICENSE file in root directory. | ||
| # Copyright(c) 2023 Intel Corporation. All Rights Reserved. | ||
|
|
||
| # test:device D400* |
Collaborator
Author
There was a problem hiding this comment.
Other are about to be deprecated, no need to add run time for them
OhadMeir
reviewed
Apr 10, 2023
|
|
||
| ################################################################################################ | ||
| test.start("Testing pipeline start/stop stress test") | ||
| for i in range(10): |
Contributor
There was a problem hiding this comment.
10 is not "stress" test. Can you add a test case that will run only on nightly context with 1000 iterations?
Collaborator
Author
There was a problem hiding this comment.
Its a mini stress test..
RealCI covers more.
The main idea is to see multiple start stop works.
1 min is enough
OhadMeir
reviewed
Apr 10, 2023
| ################################################################################################ | ||
| test.start("Testing pipeline start/stop stress test") | ||
| for i in range(10): | ||
| log.out("starting iteration #", i, "/", 10) |
Contributor
There was a problem hiding this comment.
Use i + 1 now last iteration is 9/10
OhadMeir
reviewed
Apr 10, 2023
| import platform | ||
|
|
||
|
|
||
| # Start depth + color streams and measure the time from stream opened until first frame arrived using pipeline API. |
Contributor
There was a problem hiding this comment.
You also run IMU, better to write Start all streams...
OhadMeir
reviewed
Apr 10, 2023
|
|
||
|
|
||
| # 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 |
OhadMeir
reviewed
Apr 10, 2023
| pipe = rs.pipeline() | ||
| start_call_stopwatch = Stopwatch() | ||
| pipe.start(config) | ||
| # wait_for_frames will through if no frames received so no assert is needed |
OhadMeir
approved these changes
Apr 10, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tracked on [LRS-751]
Test result