diff options
| author | Sang-Hoon Park <sang-hoon.park@arm.com> | 2021-04-13 20:21:11 +0100 |
|---|---|---|
| committer | Georgios Pinitas <georgios.pinitas@arm.com> | 2021-04-19 07:31:51 +0000 |
| commit | fe56edb4fd7a620fea4b6002d87a9763bdf8791a (patch) | |
| tree | 62615f99b01b5c5811eb92c77cc729bbf7bd3d9a /tests/validation/fixtures/PoolingLayerFixture.h | |
| parent | 9a81cd82a8102ee0bd69bfe4939d5c867aed15e9 (diff) | |
| download | ComputeLibrary-fe56edb4fd7a620fea4b6002d87a9763bdf8791a.tar.gz | |
Add padding consideration to pooling index computation
Fix the pooling kernel which has been missing consideration
of left padding, which can be implictly added by external
kernels.
Additionally, tests for FP16 have been added for the logic.
Resolves: COMPMID-4363
Change-Id: I5655991cb80f749fb1ae9bbd3918b436a078f5d1
Signed-off-by: Sang-Hoon Park <sang-hoon.park@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/5421
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'tests/validation/fixtures/PoolingLayerFixture.h')
| -rw-r--r-- | tests/validation/fixtures/PoolingLayerFixture.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/validation/fixtures/PoolingLayerFixture.h b/tests/validation/fixtures/PoolingLayerFixture.h index 66e09d5bdb..74ba3b3431 100644 --- a/tests/validation/fixtures/PoolingLayerFixture.h +++ b/tests/validation/fixtures/PoolingLayerFixture.h @@ -114,8 +114,7 @@ protected: ARM_COMPUTE_EXPECT(dst.info()->is_resizable(), framework::LogLevel::ERRORS); ARM_COMPUTE_EXPECT(_target_indices.info()->is_resizable(), framework::LogLevel::ERRORS); - // TODO: uncomment after COMPMID-4363 - // add_padding_x({ &src, &dst, &_target_indices }, data_layout); + add_padding_x({ &src, &dst, &_target_indices }, data_layout); // Allocate tensors src.allocator()->allocate(); |