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 over 3 years ago. Updated about 1 year 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 almost 3 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 over 1 year ago

  • Priority changed from High to Normal
Actions #4

Updated by bruno katra about 1 year 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