commit 4bce11548268b9a10b0f06eff3cff9ae06d5b9dd
author Georgios Pinitas <georgios.pinitas@arm.com> Wed Feb 05 16:22:14 2020 +0000
committer Georgios Pinitas <georgios.pinitas@arm.com> Wed Feb 05 21:14:13 2020 +0000
tree c5f317521d44c70cfd568ec19bb4f5a5b987e94f
parent f30318adafc9825b785738930ad239e74d227a42

COMPMID-3063: Address Custom Convolution failures

Disallows generation of scales equal to 0 on CustomConvolution fixture
as it is expected to fail according to the OpenVX specification.

Signed-off-by: Georgios Pinitas <georgios.pinitas@arm.com>
Change-Id: Ie1311c1ebbb34c72dee62da8bfc6016c4e05b83e
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/2687
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>

tests/validation/fixtures/ConvolutionFixture.h

diff --git a/tests/validation/fixtures/ConvolutionFixture.h b/tests/validation/fixtures/ConvolutionFixture.h
index 8877bd1..04172a6 100644
--- a/tests/validation/fixtures/ConvolutionFixture.h
+++ b/tests/validation/fixtures/ConvolutionFixture.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2017-2019 ARM Limited.
+ * Copyright (c) 2017-2020 ARM Limited.
  *
  * SPDX-License-Identifier: MIT
  *
@@ -50,10 +50,11 @@
     {
         std::mt19937                           gen(library->seed());
         std::uniform_int_distribution<uint8_t> distribution(0, 255);
+        std::uniform_int_distribution<uint8_t> scale_distribution(1, 255);
         const uint8_t                          constant_border_value = distribution(gen);
 
-        // Generate random scale value between 0 and 255.
-        const uint32_t scale = distribution(gen);
+        // Generate random scale value between 1 and 255.
+        const uint32_t scale = scale_distribution(gen);
 
         ARM_COMPUTE_ERROR_ON(3 != width && 5 != width && 7 != width && 9 != width);
         ARM_COMPUTE_ERROR_ON(3 != height && 5 != height && 7 != height && 9 != height);