Enable tests for D555 #14056
Enable tests for D555 #14056
Conversation
| @@ -26,8 +27,11 @@ | |||
| if product_line == "D400": | |||
| max_delay_for_depth_frame = 1 | |||
| max_delay_for_color_frame = 1 | |||
| elif product_line == "D500": | |||
| max_delay_for_depth_frame = 1 | |||
| max_delay_for_color_frame = 1 | |||
There was a problem hiding this comment.
Pipeline start take less than 1 sec for D555?
| @@ -15,6 +16,7 @@ | |||
| device_removed = False | |||
| device_added = False | |||
| MAX_ENUM_TIME_D400 = 5 # [sec] | |||
| MAX_ENUM_TIME_D500 = 18 # [sec] | |||
There was a problem hiding this comment.
@OhadMeir 18 sec is acceptable?
There was a problem hiding this comment.
We have seen in Wireshark that D555 sometimes goes back up after 13 seconds. Giving a few additional seconds for discovery and initialization 18 seems OK.
You might want to limit it based on connection type? USB should be faster. Don't know how long it would take for future D500 products...
There was a problem hiding this comment.
Let's do the timing based on connection type like Ohad suggested please
| @@ -3,6 +3,7 @@ | |||
|
|
|||
|
|
|||
| #test:device each(D400*) !D457 | |||
| #test:device each(D500*) !D555 | |||
There was a problem hiding this comment.
Will this pass without finding a D500 device?
There was a problem hiding this comment.
Yes - tested with just a D400 connected and it passed
| @@ -31,6 +33,8 @@ def device_changed( info ): | |||
| def get_max_enum_rime_by_device( dev ): | |||
|
|
Tracked on: [LRS-1253]