Project

General

Profile

Bug #62

Mise à jour ICD

Added by Gerald Saule over 7 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Start date:
21/02/2014
Due date:
% Done:

0%

Estimated time:
revision:
r0

Description

Cette issue rassemble toutes les modifications à apporter pour la prochaine ICD.
Cette issue reprend l'issue Bug#1008 ('Mise à jour ICD') de pc-instru.
Elle fait suite à Bug 816 fermée le 13/12/2013.

Dans les TM_LFR_HK, il serait très utile d'avoir deux champs complémentaires: pointeur de pile (32bits) et pointeur de tas (32bits). Cela permettrait de contrôler d'éventuels pb respectivement de récursivité, et de perte mémoire.
L'absence d'unités constitue un risque d'incompréhension. Sont concernés notamment les champs en "temperature", et "voltage".
Les LSB et offsets donnés dans les tableaux "débordent" parfois sur les cases adjacentes (dans les pdf livrés). A priori, les valeurs sont exploitables même dans les cas où les chiffres recouvrent entièrement les deux colonnes "LSB" et "Offset". Toutefois, il serait plus sain d'éviter ces débordements.
Il y a une dysymétrie pour TC_LFR_UPDATE_INFO: le LSB de HK_BIA_BIAS1 est à 0,0136646, pour HK_BIA_BIAS2 et HK_BIA_BIAS3, il est à 0,01366459. Bien que l'écart soit numériquement infime, n'y a-t'il pas un problème de principe ?
Pour chaque champ, on comprend que la grandeur physique vaut la valeur du champ * LSB avec un offset; la formule est-elle bien GrandeurPhysique = ValeurChamp * LSB + Offset ? (et pas GrandeurPhysique = ValeurChamp * LSB - Offset).
Les tableaux utilisent le tiret "-" pour remplir les cases vides. Certains tirets cohabitent avec des valeurs numériques. Ainsi, il serait prudent de confirmer que les valeurs négatives écrites sont bien intentionnelles. Il y a un risque d'oubli de tirets résiduels rendant malencontreusement une valeur négative.
Les barres de modifications pourraient être utilisées pour mettre en évidences les changements depuis la version 2.0.
Pour chaque TC/TM, la partie PACKET_SEQUENCE_CONTROL décrit le champ SEGMENTATION_GROUPING_FLAG. Cet Enum devrait être restreint à "STANDALONE_PACKET = 3", depuis la suppression de "CONTINUATION_PACKET = 0", "FIRST_PACKET_OF_A_GROUP_OF_PACKETS = 1", ""LAST_PACKET_OF_A_GROUP_OF_PACKETS = 2".
Dans les PACKET_DATA_FIELD: SOURCE_DATA, pour TM_LFR_SCIENCE_NORMAL_CWF_F3 , le champ PA_LFR_CWF3_BLK_NR a une dénomination inhomogène par rapport aux autres TM_LFR_SCIENCE.
Pour TM_LFR_SCIENCE_NORMAL_ASM_F1 Packet, la valeur min du champs PA_LFR_ASM_F1_BLK_NR est 1. Cela est contradictoire avec la ligne suivante ("/SOURCE_DATA/Repeated N times, with N in [0..52["). (idem pour PA_LFR_ASM_F2_BLK_NR du TM_LFR_SCIENCE_NORMAL_ASM_F2 Packet).
Les valeurs de l'Enumm PA_RPW_TC_FAILURE_CODE pourraient être supprimées, conformément à SSS-CP-EQS-150.
ILLEGAL_APID = 0 - Illegal APID (PAC error)
WRONG_LEN_PKT = 1 - Incomplete or invalid length packet
INCOR_CHECKSUM = 2 - Incorrect checksum
ILL_TYPE = 3 - Illegal packet type
ILL_SUBTYPE = 4 - Illegal packet subtype
WRONG_SRC_ID = 42001 - wrong source ID
NOT_ALLOWED = 42004 - failure or malfunction detected

Biblio: RPW-SYS-MEB-LFR-ICD-00097-LES_Issue2_Rev0(RPW_ICD_LFR_TC_TM)


Related issues

Related to Bug #65: TC_LFR_LOAD_NORMAL_PAR: pas de vérif sur SY_LFR_N_ASM_P, S_LFR_N_BP_P0, SY_LFR_N_BP_P1.Closed2014-02-24

Related to Feature #481: Cohérence/Intégrité sur TC_LFR_LOAD_NORMAL_PARClosed2015-08-10

History

#1 Updated by paul leroy over 7 years ago

  • Status changed from New to In Progress

Si on se réfère à la spécification LFR, RPW-MEB-LFR-00003, la valeur par défaut du paramètre SY_LFR_S1_BP_P0 est 0.25 s, et non 1 s.

==> ICD 3.11 (probablement avant aussi) spécifie 0.25 entre parenthèse mais encore pas assez explicite.

#2 Updated by Gerald Saule over 7 years ago

La note #1 montre l'ambiguïté des valeurs par défaut pour tout champ avec le LSB<>1; la valeur de l'ICD est-elle la valeur du champ transmis, ou la valeur numérique de la grandeur (cas avec SY_LFR_S1_BP_P0 de TM_LFR_PARAMETER_DUMP). L'issue Bug #54 (TM_LFR_PARAMETER_DUMP: affectation de SY_LFR_S1_BP_P0 par défaut) traite ce point.

#3 Updated by Gerald Saule over 7 years ago

SY_LFR_N_CWF_LONG_F3 du TC_LFR_LOAD_NORMAL_PAR packet n'a pas de valeur par défaut dans RPW-SYS-MEB-LFR-ICD-00097_Issue2-Rev0 (du Jan. 17, 2014).
Il faudrait mettre 0, par cohérence avec la SSS "The nominal case is to transmit TM_LFR_SCIENCE_NORMAL_CWF_F3 packets" en note du SSS-CP-EQS-470 au §4.5.10.2 LFR DATA PACKETS (RPW-SYS-SSS-00013-LES_Issue2-Rev1 du 20/01/2014).

==> Pris en compte dans ICD 3.11 (probablement avant aussi)

#4 Updated by bruno katra over 7 years ago

La contrainte d'avoir SY_LFR_N_SWP_P égal à un multiple de 8 n'est plus d'actualité (cf. Paul issue #82) : il faut faire modifier l'ICD pour mettre comme valeur max à SY_LFR_N_SWP_P :
65535 au lieu de 65528.
==> demandé dans https://jira-lesia.obspm.fr/browse/RPWIDB-162

#5 Updated by paul leroy over 6 years ago

  • Status changed from In Progress to Feedback
  • Assignee changed from paul leroy to Veronique bouzid

Il faudrait repointer les informations, s'il y en a encore, à faire remonter dans l'ICD.

sy_lfr_n_swf_p peut valoir de 16 22 à 65535 (repris de la remarque ci-dessus non prise en compte dans l'ICD 3.9)
(passage à 22s depuis modfis FSW pour resynchro SWF afin d'éviter des erreurs, devra aussi être géré par l'envoi d'un TM_EXE_TC par le FSW)
==> demandé dans https://jira-lesia.obspm.fr/browse/RPWIDB-162

#6 Updated by Veronique bouzid about 6 years ago

  • Status changed from Feedback to Closed

voir Feature #481 qui décrit les tests effectués sur les paramètres
==> demandé dans https://jira-lesia.obspm.fr/browse/RPWIDB-162

#7 Updated by bruno katra about 6 years ago

  • Status changed from Closed to Feedback

#8 Updated by bruno katra over 5 years ago

Les valeurs ASM sont marquée comme integer dans l'iCD alors que ce sont des floats IEEE754
==> demandé dans https://jira-lesia.obspm.fr/browse/RPWIDB-162

#9 Updated by bruno katra over 5 years ago

Nombreuses demandes de modifs faites dans :
https://jira-lesia.obspm.fr/browse/RPWIDB-162

#10 Updated by Veronique bouzid over 4 years ago

  • Status changed from Feedback to Closed

Also available in: Atom PDF