From 0eedf7f68ef40c185b15597faeebc7b756859329 Mon Sep 17 00:00:00 2001
From: Olivier BICHLER <olivier.bichler@cea.fr>
Date: Mon, 3 Jun 2024 14:05:46 +0200
Subject: [PATCH] Improved "not implemented" errors
---
include/aidge/operator/MetaOperator.hpp | 2 +-
src/backend/OperatorImpl.cpp | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/aidge/operator/MetaOperator.hpp b/include/aidge/operator/MetaOperator.hpp
index a41110161..75c4d886f 100644
--- a/include/aidge/operator/MetaOperator.hpp
+++ b/include/aidge/operator/MetaOperator.hpp
@@ -118,7 +118,7 @@ public:
void updateConsummerProducer() override;
void forward() override;
void backward() override {
- assert(false && "not implemented");
+ AIDGE_THROW_OR_ABORT(std::runtime_error, "backward() not implemented yet for a MetaOperator");
}
inline bool isAtomic() const noexcept override final { return false; }
diff --git a/src/backend/OperatorImpl.cpp b/src/backend/OperatorImpl.cpp
index 2277db242..8a5b40e44 100644
--- a/src/backend/OperatorImpl.cpp
+++ b/src/backend/OperatorImpl.cpp
@@ -129,9 +129,9 @@ void Aidge::OperatorImpl::resetConsummerProducer(){
}
void Aidge::OperatorImpl::forward() {
- AIDGE_THROW_OR_ABORT(std::runtime_error, "forward() not implemented");
+ AIDGE_THROW_OR_ABORT(std::runtime_error, "forward() not implemented yet for operator of type {}", mOp.type());
}
void Aidge::OperatorImpl::backward() {
- AIDGE_THROW_OR_ABORT(std::runtime_error, "backward() not implemented");
+ AIDGE_THROW_OR_ABORT(std::runtime_error, "backward() not implemented yet for operator of type {}", mOp.type());
}
--
GitLab