From 3668481fbe6a3fafb22a96e77361fabed4a439fd Mon Sep 17 00:00:00 2001 From: Olivier BICHLER <olivier.bichler@cea.fr> Date: Wed, 2 Jul 2025 12:38:53 +0200 Subject: [PATCH] Fixed saturation value in activation_forward_value() --- aidge_export_cpp/static/activation_utils.hpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/aidge_export_cpp/static/activation_utils.hpp b/aidge_export_cpp/static/activation_utils.hpp index c6a1bcd..ddac7fb 100644 --- a/aidge_export_cpp/static/activation_utils.hpp +++ b/aidge_export_cpp/static/activation_utils.hpp @@ -48,9 +48,5 @@ Output_T activation_forward_value (Sum_T weightedSum, break; } - // Value fixed here for now but it should be generated by - // the export module or determined by the type of Output_T - // For now only works for int8_t and uint8_t - const uint32_t NB_BITS = 8; - return saturate<Output_T>(rescaling(weightedSum, output), NB_BITS); + return saturate<Output_T>(rescaling(weightedSum, output), 8 * sizeof(Output_T)); } -- GitLab