From 3e05e4e85912e745b8555102e1bcef13478d2ceb Mon Sep 17 00:00:00 2001 From: steniu01 Date: Fri, 25 Aug 2017 17:18:01 +0100 Subject: COMPMID-516 Change the CL CNN validation functions to use relative tolerance error Change-Id: Iec6347af26ea2a83c911f5fe10e6048e8a2a47ba Reviewed-on: http://mpd-gerrit.cambridge.arm.com/85381 Tested-by: Kaizen Reviewed-by: Moritz Pflanzer --- tests/validation/CL/FullyConnectedLayer.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'tests/validation/CL/FullyConnectedLayer.cpp') diff --git a/tests/validation/CL/FullyConnectedLayer.cpp b/tests/validation/CL/FullyConnectedLayer.cpp index 7a8734b5a3..356e9677fc 100644 --- a/tests/validation/CL/FullyConnectedLayer.cpp +++ b/tests/validation/CL/FullyConnectedLayer.cpp @@ -44,8 +44,10 @@ namespace validation namespace { /** Tolerance for float operations */ -constexpr AbsoluteTolerance tolerance_f32(0.001f); -constexpr AbsoluteTolerance tolerance_f16(0.4f); +RelativeTolerance tolerance_f32(0.001f); +RelativeTolerance tolerance_f16(half_float::half(0.2)); +constexpr float tolerance_num = 0.07f; /**< Tolerance number */ + /** Tolerance for fixed point operations */ constexpr AbsoluteTolerance tolerance_fixed_point(1.f); @@ -112,14 +114,14 @@ FIXTURE_DATA_TEST_CASE(RunSmall, CLFullyConnectedLayerFixture, framework::dataset::make("DataType", DataType::F16))) { // Validate output - validate(CLAccessor(_target), _reference, tolerance_f16); + validate(CLAccessor(_target), _reference, tolerance_f16, tolerance_num); } FIXTURE_DATA_TEST_CASE(RunLarge, CLFullyConnectedLayerFixture, framework::DatasetMode::NIGHTLY, combine(combine(datasets::LargeFullyConnectedLayerDataset(), FullyConnectedParameters), framework::dataset::make("DataType", DataType::F16))) { // Validate output - validate(CLAccessor(_target), _reference, tolerance_f16); + validate(CLAccessor(_target), _reference, tolerance_f16, tolerance_num); } TEST_SUITE_END() -- cgit v1.2.1