From: Manuel Bottini <manuel.bottini@arm.com>
Date: Tue, 17 Nov 2020 12:30:38 +0000 (+0000)
Subject: COMPMID-3971: Nightly failure - CL LaplacianPyramid mismatches
X-Git-Tag: submit/tizen/20210421.062230~215
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=10a73f5c0cc15ffba532bc923c6471f67af0959a;p=platform%2Fupstream%2Farmcl.git

COMPMID-3971: Nightly failure - CL LaplacianPyramid mismatches

Change-Id: I8b7dee38d30a7aeb1770cf1cf592c2b5e3d3f93d
Signed-off-by: Manuel Bottini <manuel.bottini@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4441
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Pablo Marquez Tello <pablo.tello@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
---

diff --git a/tests/validation/CL/LaplacianPyramid.cpp b/tests/validation/CL/LaplacianPyramid.cpp
index 3096d3297..78f3f2373 100644
--- a/tests/validation/CL/LaplacianPyramid.cpp
+++ b/tests/validation/CL/LaplacianPyramid.cpp
@@ -43,8 +43,12 @@ namespace validation
 {
 namespace
 {
-const auto small_laplacian_pyramid_levels = framework::dataset::make("NumLevels", 2, 3);
-const auto large_laplacian_pyramid_levels = framework::dataset::make("NumLevels", 2, 5);
+/* Absolute tolerance value for comparing reference's output against implementation's output for DataType::S16
+ * Tolerance is needed for calculation uncertainties introduced from the layers
+ */
+AbsoluteTolerance<int16_t> tolerance_int16(1);
+const auto                 small_laplacian_pyramid_levels = framework::dataset::make("NumLevels", 2, 3);
+const auto                 large_laplacian_pyramid_levels = framework::dataset::make("NumLevels", 2, 5);
 
 const auto formats = combine(framework::dataset::make("FormatIn", Format::U8), framework::dataset::make("FormatOut", Format::S16));
 
@@ -68,7 +72,7 @@ inline void validate_laplacian_pyramid(const CLPyramid &target, const std::vecto
                                                                border_mode == BorderMode::UNDEFINED);
 
         // Validate level
-        validate(CLAccessor(*level_image), reference[lev], valid_region);
+        validate(CLAccessor(*level_image), reference[lev], valid_region, tolerance_int16);
     }
 }
 } // namespace