Project

General

Profile

Actions

Feature #3905

open

La TM_KCOEFF_DUMP n'a plus la structure dédcrite dans l'ICD

Added by bruno katra 2 months ago. Updated 17 days ago.

Status:
In Progress
Priority:
High
Assignee:
Category:
-
Target version:
Start date:
24/11/2021
Due date:
% Done:

0%

Estimated time:

Description

L'ICD précise qu'un KCOEFF_DUMP contient :

n blocs de [KCOEFF_FREQ + 32xKCOEFF] où KCOEFF_FREQ est un UINT16 et KCOEFF des floats.

Or dans FSW > 3.3 : le paramètre KCOEFF_FREQ n'a plus de sens et son espace est utilisé pour stocker directement des floats.

Les conséquences :
- Il faut mettre à jour la decom LFR (Bruno) : en cours fait!
- Le bouton du LFR GSE pour dumper les KCOEFF n'est plus utilisable car elle décommute à la volée selon l'ICD et donc lit 1 float sur 32 comme un entier 16 bits
- Le champ BLOCK_NUMBER qui est 30 ou 6 (selon que c'est le 1er ou le 2eme paquet de KCOEFF_DUMP) n'est plus applicable, on pourrait le modifier pour qu'il donne le nombre de float à lire (changer les DEFINE dans le code du FSW) : Alexis > FAIT
Il va falloir prévenir Plasson ou Leeroy pour modifier l'ICD avec les remarques ci-dessus car le segment sol du LESIA l'utilise pour nous générer les L1 : Bruno


Related issues

Related to Bug #3908: Taille des paquets KCOEFF_DUMP en FSW >=3.3Closedbruno katra26/11/2021

Actions
Related to Bug #3911: KCOEFF_DUMP : champ PA_LFR_KCOEFF_BLK_NR fauxClosedbruno katra02/12/2021

Actions
Related to Bug #3916: Les valeurs reportées dans le KCOEFF_DUMP ne sont pas comme attendues.In ProgressAlexis Jeandet08/12/2021

Actions
Related to Task #3918: Mise à jour des 2 DEFINE PA_LFR_KCOEFF_BLK_NR pour le KCOEFF_DUMP ClosedAlexis Jeandet11/01/2022

Actions
Actions #1

Updated by bruno katra 2 months ago

  • Priority changed from Normal to High
Actions #2

Updated by bruno katra about 2 months ago

  • Related to Bug #3908: Taille des paquets KCOEFF_DUMP en FSW >=3.3 added
Actions #3

Updated by bruno katra about 2 months ago

  • Description updated (diff)
  • Assignee changed from bruno katra to Alexis Jeandet
  • Priority changed from High to Urgent

ll faudrait changer les 2 DEFINE du code du FSW qui contiennent le champ BLOCK_NUMBER : Les DEFINE sont actuellement à 30 et 6, il faudrait les mettre à 676 et 338.
Cela devrait résoudre l'issue 3908.

Actions #4

Updated by bruno katra about 2 months ago

  • Priority changed from Urgent to High
Actions #5

Updated by bruno katra about 2 months ago

  • Status changed from New to In Progress

Pb #3911 : le champs blk_number est un octet, on ne peut pas stocker 676 et 338.

Discuté avec Alexis : le BLOCK_NUMBER pourrait contenir le nombre de floats/26 (26= nombre de floats pour 1matrice B + 1 matrice E).
L'ICD serait modifié avec juste la modification du champs en jaune :

/SOURCE_DATA/Repeated N times, with N in [0..PA_LFR_KCOEFF_BLK_NR[

deviendrait
/SOURCE_DATA/Repeated N times, with N in [0..PA_LFR_KCOEFF_BLK_NR*26[

+

/SOURCE_DATA/[...]/LFR_KCOEFFICIENTS_PARAMETERS qui contient actuellement une structure 1UINT16 + 32 floats

deviendrait juste 1 float

A proposer à Plasson + Xavier

Actions #6

Updated by bruno katra about 2 months ago

  • Related to Bug #3911: KCOEFF_DUMP : champ PA_LFR_KCOEFF_BLK_NR faux added
Actions #7

Updated by bruno katra about 2 months ago

  • Description updated (diff)
Actions #8

Updated by bruno katra about 2 months ago

  • Assignee changed from Alexis Jeandet to bruno katra

Il faut maintenant prévenir le LESIA

Actions #9

Updated by bruno katra 17 days ago

  • Related to Bug #3916: Les valeurs reportées dans le KCOEFF_DUMP ne sont pas comme attendues. added
Actions #10

Updated by bruno katra 17 days ago

Partant de la résolution de #3916, voici une nouvelles proposition "plus élégante" pour la mise à jour de l'ICD et la définition du champs PA_LFR_KCOEFF_BLK_NR :

le champs PA_LFR_KCOEFF_BLK_NR pourrait contenir le nombre de couple de matrices B/E pour un bin donné.

Il serait donc égal à 26 pour le 1er paquet et 13 pour le 2ème : ça tient dans un uint8.

Pour Alexis cela revient juste à changer les 2 DEFINE dans le code.

Côté ICD, il ne faudrait modifier que la partie :

/SOURCE_DATA/[...]/LFR_KCOEFFICIENTS_PARAMETERS qui contient actuellement une structure 1UINT16 + 32 floats

et qui deviendrait :

B11_real FLOAT
B11_ima FLOAT
B12_real FLOAT
B12_imag FLOAT
....
B33_ima FLOAT
E11_real FLOAT
E11_ima FLOAT
....
E22_ima FLOAT

Cette représentation est plus pertinente et plus structurée qu'une suite de floats.
Au niveau L1 cela permettra des CDF plus pertinent et auto consistants.

A proposer à Philippe Plasson et Leeroy.

Actions #11

Updated by bruno katra 17 days ago

  • Related to Task #3918: Mise à jour des 2 DEFINE PA_LFR_KCOEFF_BLK_NR pour le KCOEFF_DUMP added
Actions

Also available in: Atom PDF