commit edc524ef7ed38e0521c874f28bb9a1f2407b44c6
author Giorgio Arena <giorgio.arena@arm.com> Wed Feb 10 11:54:47 2021 +0000
committer Giorgio Arena <giorgio.arena@arm.com> Wed Feb 10 16:10:32 2021 +0000
tree 64eab5750f0284ddc184f44a5e5b6e8b7472d59e
parent db35345753e4ba81384c8a92ece6a8f598fd841a

Revert changes on tensor's strides and fix CLDepthwiseConvolution 3x3 Quantized

- Revert changes in strides > num_dimensions. Set them to 0
- Fix offset calculcation in depthwise 3x3 quantized using select and stride_y for max offset

Resolve COMPMID-4254

Change-Id: Ia99b9637f18b99b1fa3d4b7b4892046027d3e7e5
Signed-off-by: Giorgio Arena <giorgio.arena@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/5040
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>

tests/validation/UNIT/TensorInfo.cpp

diff --git a/tests/validation/UNIT/TensorInfo.cpp b/tests/validation/UNIT/TensorInfo.cpp
index 44c9342..cf9dfea 100644
--- a/tests/validation/UNIT/TensorInfo.cpp
+++ b/tests/validation/UNIT/TensorInfo.cpp
@@ -60,11 +60,11 @@
                PaddingSize{ 4, 36, 4, 4 }})),
                framework::dataset::make("Strides", {
                Strides{},
-               Strides{ 1U, 50U, 50U, 50U, 50U, 50U },
-               Strides{ 1U, 50U, 900U, 900U, 900U, 900U },
-               Strides{ 1U, 50U, 900U, 900U, 900U, 900U },
-               Strides{ 1U, 50U, 900U, 9000U, 9000U, 9000U },
-               Strides{ 1U, 50U, 900U, 9000U, 90000U, 90000U },
+               Strides{ 1U, 50U },
+               Strides{ 1U, 50U },
+               Strides{ 1U, 50U, 900U },
+               Strides{ 1U, 50U, 900U, 9000U },
+               Strides{ 1U, 50U, 900U, 9000U, 90000U },
                Strides{ 1U, 50U, 900U, 9000U, 90000U, 900000U }})),
                framework::dataset::make("Offset", { 0U, 4U, 204U, 204U, 204U, 204U, 204U })),
                shape, auto_padding, strides, offset)