From f30d7413b2ba6625dc0f86ca540694a2a349dd54 Mon Sep 17 00:00:00 2001 From: Liam Barry Date: Wed, 27 Sep 2023 12:48:35 +0100 Subject: MLECO-4365: Add CI Test for insufficient tensor arena and update common tests. Previously, execution path following failed tensor allocation was not covered by tests. Minor amendment added to make ~Model virtual. During testing it was noticed that intentional errors simulated in certain common tests would result in misleading output for subsequent tests as the error messages appear in the console output of the next failing test. Documentation and print statements added until a solution is found. Change-Id: Iaf3e0f32ce9e0505921251cd41f73030108d2cb1 Signed-off-by: Liam Barry --- source/application/api/common/include/Model.hpp | 2 +- source/application/api/common/source/Model.cc | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) (limited to 'source') diff --git a/source/application/api/common/include/Model.hpp b/source/application/api/common/include/Model.hpp index 1728e1f..6eefd02 100644 --- a/source/application/api/common/include/Model.hpp +++ b/source/application/api/common/include/Model.hpp @@ -33,7 +33,7 @@ namespace app { Model(); /** @brief Destructor. */ - ~Model(); + virtual ~Model() = default; /** @brief Gets the pointer to the model's input tensor at given input index. */ TfLiteTensor* GetInputTensor(size_t index) const; diff --git a/source/application/api/common/source/Model.cc b/source/application/api/common/source/Model.cc index b344a53..3675c5b 100644 --- a/source/application/api/common/source/Model.cc +++ b/source/application/api/common/source/Model.cc @@ -20,8 +20,6 @@ #include #include -arm::app::Model::~Model() = default; - arm::app::Model::Model() : m_inited(false), m_type(kTfLiteNoType) {} /* Initialise the model */ -- cgit v1.2.1