From fc1ffe626278e75ba11c803280a6ef46a5bd1ad6 Mon Sep 17 00:00:00 2001 From: Pablo Tello Date: Tue, 23 Jan 2018 08:31:41 +0000 Subject: COMPMID-837: Fixed remap tests failures in Valgrind. Some minor improvements in the test fixture, for example making sure the values in the mapx and mapy tensors are in the range of [-5, in_width+5] and [-5,in_height]. Tolerance was changed to 0, no mismatches expected. Change-Id: I2fad06defb293bf9fdd1988799b19547c102dee5 Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/118044 Tested-by: Jenkins Reviewed-by: Georgios Pinitas Reviewed-by: Anthony Barbier --- tests/validation/NEON/Remap.cpp | 6 +++--- tests/validation/fixtures/RemapFixture.h | 21 +++++++++++---------- tests/validation/reference/Remap.cpp | 19 ++++++++++++------- 3 files changed, 26 insertions(+), 20 deletions(-) (limited to 'tests/validation') diff --git a/tests/validation/NEON/Remap.cpp b/tests/validation/NEON/Remap.cpp index 6e58000d54..2e54b1152e 100644 --- a/tests/validation/NEON/Remap.cpp +++ b/tests/validation/NEON/Remap.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017 ARM Limited. + * Copyright (c) 2017-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -43,8 +43,8 @@ namespace validation { namespace { -constexpr AbsoluteTolerance tolerance_value(1); -constexpr float tolerance_number = 0.2f; +constexpr AbsoluteTolerance tolerance_value(0); +constexpr float tolerance_number = 0.f; } // namespace TEST_SUITE(NEON) diff --git a/tests/validation/fixtures/RemapFixture.h b/tests/validation/fixtures/RemapFixture.h index 846ebf44a2..78b30151ac 100644 --- a/tests/validation/fixtures/RemapFixture.h +++ b/tests/validation/fixtures/RemapFixture.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017 ARM Limited. + * Copyright (c) 2017-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -50,7 +50,7 @@ public: { std::mt19937 gen(library->seed()); std::uniform_int_distribution distribution(0, 255); - T constant_border_value = static_cast(distribution(gen)); + const T constant_border_value = static_cast(distribution(gen)); _target = compute_target(shape, policy, data_type, border_mode, constant_border_value); _reference = compute_reference(shape, policy, data_type, border_mode, constant_border_value); @@ -58,9 +58,10 @@ public: protected: template - void fill(U &&tensor, int i) + void fill(U &&tensor, int i, float min, float max) { - library->fill_tensor_uniform(tensor, i); + std::uniform_int_distribution<> distribution((int)min, (int)max); + library->fill(tensor, distribution, i); } TensorType compute_target(const TensorShape &shape, InterpolationPolicy policy, DataType data_type, BorderMode border_mode, T constant_border_value) @@ -92,9 +93,9 @@ protected: ARM_COMPUTE_EXPECT(!dst.info()->is_resizable(), framework::LogLevel::ERRORS); // Fill tensors - fill(AccessorType(src), 0); - fill(AccessorType(map_x), 1); - fill(AccessorType(map_y), 2); + fill(AccessorType(src), 0, 0, 255); + fill(AccessorType(map_x), 1, -5, shape.x() + 5); + fill(AccessorType(map_y), 2, -5, shape.y() + 5); // Compute function remap.run(); @@ -115,9 +116,9 @@ protected: _valid_mask = SimpleTensor { shape, data_type }; // Fill reference - fill(src, 0); - fill(map_x, 1); - fill(map_y, 2); + fill(src, 0, 0, 255); + fill(map_x, 1, -5, shape.x() + 5); + fill(map_y, 2, -5, shape.y() + 5); // Compute reference return reference::remap(src, map_x, map_y, _valid_mask, policy, border_mode, constant_border_value); diff --git a/tests/validation/reference/Remap.cpp b/tests/validation/reference/Remap.cpp index bef5962fbf..f862c13700 100644 --- a/tests/validation/reference/Remap.cpp +++ b/tests/validation/reference/Remap.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017 ARM Limited. + * Copyright (c) 2017-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -42,16 +42,15 @@ SimpleTensor remap(const SimpleTensor &in, SimpleTensor &map_x, Sim T constant_border_value) { ARM_COMPUTE_ERROR_ON_MSG(border_mode == BorderMode::REPLICATE, "BorderMode not supported"); - SimpleTensor out(in.shape(), in.data_type()); - + ARM_COMPUTE_ERROR_ON(out.num_elements() != map_x.num_elements()); const int width = in.shape().x(); const int height = in.shape().y(); - for(int idx = 0; idx < out.num_elements(); idx++) { - valid_mask[idx] = 1; - Coordinates src_idx; + const Coordinates id_out = index2coord(out.shape(), idx); + valid_mask[idx] = 1; + Coordinates src_idx = id_out; // need to setup all coordinates and not just xy src_idx.set(0, static_cast(std::floor(map_x[idx]))); src_idx.set(1, static_cast(std::floor(map_y[idx]))); if((0 <= map_y[idx]) && (map_y[idx] < height) && (0 <= map_x[idx]) && (map_x[idx] < width)) @@ -59,11 +58,17 @@ SimpleTensor remap(const SimpleTensor &in, SimpleTensor &map_x, Sim switch(policy) { case InterpolationPolicy::NEAREST_NEIGHBOR: + { out[idx] = tensor_elem_at(in, src_idx, border_mode, constant_border_value); break; + } case InterpolationPolicy::BILINEAR: - (valid_bilinear_policy(map_x[idx], map_y[idx], width, height, border_mode)) ? out[idx] = bilinear_policy(in, src_idx, map_x[idx], map_y[idx], border_mode, constant_border_value) : valid_mask[idx] = 0; + { + (valid_bilinear_policy(map_x[idx], map_y[idx], width, height, border_mode)) ? + out[idx] = bilinear_policy(in, src_idx, map_x[idx], map_y[idx], border_mode, constant_border_value) : + valid_mask[idx] = 0; break; + } case InterpolationPolicy::AREA: default: ARM_COMPUTE_ERROR("Interpolation not supported"); -- cgit v1.2.1