Project

General

Profile

Actions

Bug #3821

open

Moyennage des SM erroné pour F0 et F1 dans FSW <=3.2.0.24

Added by bruno katra almost 3 years ago. Updated 10 months ago.

Status:
Resolved
Priority:
Normal
Category:
-
Target version:
Start date:
29/03/2021
Due date:
% Done:

0%

Estimated time:
revision:
r0

Description

Pour F0 et F1, on moyenne 8 SM pour faire une ASM. La boucle est fausse : Seul la première SM est comptabilisée et moyennée (*8/8).

https://github.com/LaboratoryOfPlasmaPhysics/LFR_Flight_Software/blame/729417ab5a3e7e5854a6665a3662e2b8ebc7756e/header/processing/fsw_processing.h#L206

Message de Alexis :

"Ce qui me choque dans la ligne en question, c'est qu'on a une boucle
sur k et qu'on somme N fois la même composante de la même matrice selon
que incomingSMIsValid[k] == MATRIX_IS_NOT_POLLUTED.
Je trouve ça étrange." 


Related issues

Related to Task #3991: Mise en place d'un protocole de test pour la validation de la correction du bug sur le moyennage des SM à F0 et F1In Progressbruno katra21/10/2022

Actions
Actions #1

Updated by Alexis Jeandet over 2 years ago

  • Status changed from New to Resolved

Fixed in this commit https://github.com/LaboratoryOfPlasmaPhysics/LFR_Flight_Software/commit/ceccb683253c98d477c8c38c9b6b676ed4c9cc6a#
This needs tests, a simple test would be to generate a short pulse at 3V each ASM period - 1s for example and check that before the fix we miss it most of the time and after we see it in each ASM.

Actions #2

Updated by bruno katra over 1 year ago

  • Related to Task #3991: Mise en place d'un protocole de test pour la validation de la correction du bug sur le moyennage des SM à F0 et F1 added
Actions #3

Updated by bruno katra about 1 year ago

  • Priority changed from High to Normal
Actions #4

Updated by bruno katra 10 months ago

Several specific tests implemented to validate the expected behaviour at SM level.
FSW 3.3.0.16 release is considered to have this bug fixed.

Actions

Also available in: Atom PDF