KhronosGroup / KhronosGroup/openvx-samples

Non-compliance with the standard

Open
#12 0 comments 0 reactions 1 assignee Claimed by @kiritigowda View on GitHub
bug
Dominant language
C++
Stars
113
Forks
26
PR merge metrics
No merged PRs in 30d

Description

There are no initializations of `dim_x` and `dim_y` in:
* https://github.com/KhronosGroup/openvx-samples/blob/master/skin-tone-detector/src/skinToneDetector.cpp#L183
* https://github.com/KhronosGroup/openvx-samples/blob/master/canny-edge-detector/src/canny.cpp#L125

But the standard says (https://registry.khronos.org/OpenVX/specs/1.3.1/html/OpenVX_Specification_1_3_1.html#:~:text=%5Bin%5D%20user_addr,REQ-0928%5D):
> In the structure, only dim_x, dim_y, stride_x and stride_y fields must be provided, other fields are ignored by the function [REQ-0927]

This requirement is in all standards.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.