search

KhronosGroup / OpenVX-cts

OpenVX Conformance Test Suite
Apache License 2.0
9 stars 6 forks source link

Incorrect status in testSwapImageHandle #17

Open RobertCall opened 4 years ago

RobertCall commented 4 years ago

Test Image.SwapImageHandle/78/_U1_/rand/sz=16x16/VX_DF_IMAGE_U1/ROI=true expected to be success. This test connected with vxMapImagePatch. But standard says:

[in] rect - The coordinates of image patch [REQ-1006]. The patch must be within the bounds of the image. (start_x, start_y) gives the coordinate of the topleft element inside the patch, while (end_x, end_y) give the coordinate of the bottomright element out of the patch. Must be 0 ≤ start < end.

But in this test rectangulars roi1 and roi2 are setted to: rect_roi1 = {8, 8, 16, 16}, rect_roi2 = {8, 4, 8, 8}. Documentation says that must be start < end. But roi_rect2.start_x == roi_rect2.end_x == 8. According to documentation this test shouldn't pass.

jessegv commented 4 years ago

@RobertCall , thanks for reporting this bug. It does look like a bug in the CTS. We will review it and push an appropriate update to the test. Thanks again.

jessegv commented 4 years ago

@RobertCall , can you confirm that the fix referenced here fixes your bug? https://github.com/KhronosGroup/OpenVX-cts/commit/f6471424a885c0684482dd6a385498dd871b81b4

RobertCall commented 4 years ago

Yes, this bug fixed