Project

General

Profile

Bug #3821

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

Added by bruno katra 6 months ago. Updated 5 days ago.

Status:
Resolved
Priority:
High
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." 

History

#1 Updated by Alexis Jeandet 5 days 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.

Also available in: Atom PDF