KhronosGroup / OpenVX-cts

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

Incorrect name in TESTCASE: GraphEnhanced #15

Closed kpanfilova closed 4 years ago

kpanfilova commented 4 years ago

The kernel names do not match the documentation according to the following table.

Link to documentation: link

Link to ctcs-test: link

Kernel name string from documentation string from cts-tests
VX_KERNEL_HOUGH_LINES_P "org.khronos.openvx.hough_lines_p" "org.khronos.openvx.hough_lines_probabilistic"
VX_KERNEL_TENSOR_MATRIX_MULTIPLY "org.khronos.openvx.tensor_matrix_multiply" "org.khronos.openvx.tensor_matrix_multiply "
VX_KERNEL_COPY "org.khronos.openvx.copy" "org.khronos.openvx.copy_node"
VX_KERNEL_NON_MAX_SUPPRESSION "org.khronos.openvx.non_max_suppression" "org.khronos.openvx.nonmaxsuppression"
VX_KERNEL_HOG_FEATURES "org.khronos.openvx.hog_features" "org.khronos.openvx.hogfeatures"
VX_KERNEL_BILATERAL_FILTER "org.khronos.openvx.bilateral_filter" "org.khronos.openvx.bilateral_filter "
VX_KERNEL_WEIGHTED_AVERAGE "org.khronos.openvx.weighted_average" "org.khronos.openvx.weightedaverage"

Which names is correct for implementation?

jessegv commented 4 years ago

Thanks. you are right. The sample and cts is not matching the spec. The spec is correct. I will update the cts and sample in internal repo and sync to github shortly.

jessegv commented 4 years ago

Fixed .. closing