https://hephaistos.lpp.polytechnique.fr/redmine/https://hephaistos.lpp.polytechnique.fr/redmine/redmine/favicon.ico?15080976012022-03-31T08:45:56ZRedmineLFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=102562022-03-31T08:45:56Zbruno katra
<ul><li><strong>Related to</strong> <i><a class="issue tracker-2 status-5 priority-3 priority-high3 closed" href="/redmine/issues/3905">Feature #3905</a>: La TM_KCOEFF_DUMP n'a plus la structure dédcrite dans l'ICD</i> added</li></ul> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=102962022-04-07T08:08:52Zbruno katra
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assignee</strong> set to <i>bruno katra</i></li></ul><p>Réponse Thomas et Philippe Plasson : cas déjà étudié en octobre 2021, pas possible de passer à 39 TC sans impact sur IDB et FSW DPU...<br />On se limitera donc à un changement de label pour essayer de garder un peu de cohérence entre TM et TC. Le KCOEFF_FREQUENCY de la TC (devenu BIN_INDEX dans la TM) ne change pas non plus, il reste un entier de 0 à 35 car pas pertinent de le passer en index des BIN puisque certaines TC contiennent les valeurs de plusieurs BIN + Leeroy dit que cela impliquera un changement d'IDB...</p> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=104412023-01-06T08:58:50Zbruno katra
<ul><li><strong>File</strong> <a href="/redmine/attachments/5472">[RID-875] Mise à jour relative à la MAJ LFR.pdf</a> <a class="icon-only icon-download" title="Download" href="/redmine/attachments/download/5472/%5BRID-875%5D%20Mise%20%C3%A0%20jour%20relative%20%C3%A0%20la%20MAJ%20LFR.pdf">[RID-875] Mise à jour relative à la MAJ LFR.pdf</a> added</li></ul><p>Mail Leeroy 8/12/2022</p>
<p>Bonjour à tous,</p>
<p>Diane, pas de soucis pour produire les TC de chargement d'ici le 27 si le logiciel est livré le 20.<br />Pour ce qui est des modifications de l'IDB, vous pouvez confirmer que ça porte au final uniquement sur les points que listé dans l'issues RID-875 dont l'export est en pièce jointe ?</p>
<p>LeeRoy</p> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=104422023-01-06T08:59:18Zbruno katra
<ul><li><strong>Assignee</strong> changed from <i>bruno katra</i> to <i>Alexis Jeandet</i></li></ul><p>Je suis en train de répondre à Leeroy, ils se sont carrément emmêlé les pinceaux dans le point JIRA. Ce qui est même bizzare c'est qu'ils parlent de faire des modifs sur les champs de la TC alors qui'ls nous avaient dit que c'était impossible... Bref...</p>
<p>Juste une question : j'ai demandé la modif du min/max du champs de la TM_KCOEFF_DUMP suivant :</p>
<p>KCOEFF_FREQ devient KCOEFF_BIN_INDEX avec un min/max de 1/128 mais j'ai l'impression que le FSW numérote à partir de 0 donc ce serait plutôt 0/127 :</p>
<p>@AlexisJ : tu confirmes?</p>
<p>Merci d'avance</p>
<p>Bruno</p> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=104432023-01-06T09:37:56ZAlexis Jeandet
<ul></ul><p>bruno katra wrote in <a href="#note-4">#note-4</a>:</p>
<blockquote>
<p>Je suis en train de répondre à Leeroy, ils se sont carrément emmêlé les pinceaux dans le point JIRA. Ce qui est même bizzare c'est qu'ils parlent de faire des modifs sur les champs de la TC alors qui'ls nous avaient dit que c'était impossible... Bref...</p>
<p>Juste une question : j'ai demandé la modif du min/max du champs de la TM_KCOEFF_DUMP suivant :</p>
<p>KCOEFF_FREQ devient KCOEFF_BIN_INDEX avec un min/max de 1/128 mais j'ai l'impression que le FSW numérote à partir de 0 donc ce serait plutôt 0/127 :</p>
<p>@AlexisJ : tu confirmes?</p>
<p>Merci d'avance</p>
<p>Bruno</p>
</blockquote>
<p>Hello,</p>
<p>Je confirme que ça part de 0:<br /><a class="external" href="https://github.com/LaboratoryOfPlasmaPhysics/LFR_Flight_Software/blob/R3.3/src/tc_tm/tc_load_dump_parameters.c#L555-L558">https://github.com/LaboratoryOfPlasmaPhysics/LFR_Flight_Software/blob/R3.3/src/tc_tm/tc_load_dump_parameters.c#L555-L558</a></p> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=104512023-01-17T13:21:01Zbruno katra
<ul></ul><p>Finalement après 10 mois de discussions, le LESIA rejette toutes les demandes de modifs (telecon 13/01/2023), voir mail ci-dessous :</p>
<p>Salut à tous, nous avons une petite teleconf rapide avec LeeRoy et Philippe cet aprem pour refaire le point suite à mon mail ci-dessous.</p>
<p>Concernant le changement des labels prévus niveau TM, Philippe estime après réflexion que cela est trop risqué en terme de régression IDB comparé au gain d'information, il est vrai léger, que ces changements apporteraient. Il préconise plutôt qu'on laisse comme c'est et que l'on fasse un tableau explicite de correspondances 'labels IDB' <-> 'nouveaux labels/fonctions' dans le SUM. Du coup on ne touche aucun label.</p>
<p>Pour la valeur min/max du champs TM KCOEFF_FREQ, le risque est moins élevé mais comme il y a un flou si cette valeur est utilisée en aval (ROC, ESA, ...), il y a toujours un risque de régression. Alexis et moi avons proposé comme solution alternative de recompiler le logiciel de vol LFR en cours de validation pour remettre des valeurs qui sont dans l'intervalle actuel donc sans modifs IDB. Philippe et LeeRoy nous ont laissé évalué si cela était acceptable (car cela entraine une nouvelle version du FSW) ou si ils tentaient la mise à jour IDB.</p>
<p>Après discussions avec Alexis : on opte pour la recompilation du FSW LFR avec des valeurs qui sont valides avec le min/max actuel de l'IDB. On prend la responsabilité de ne pas recommencer la campagne de tests intégralement car la modif est plus que minime : on remplace la variable KCOEFF_FREQ (UINT_16) actuelle qui oscille entre 0 et 127 par : constante '0' pour les matrices de passage F0, constante '1' pour les matrices de passage F1,constante '2' pour les matrices de passage F2. Je rejouerai quand même les tests spécifiques sur les KCOEFF puisque c'est la seule portion du code qui est touchée.</p>
<p>Diane et Xavier : pour ce qui est de la livraison prévue le 20/01, nous avons pris du retard suite aux diverses grippes hivernales et à un problème de régression pas sur LFR mais sur le banc de test! (les machines vieillissent ) Le problème semble résolu depuis cette semaine et on a repris le rythme de croisière. Pour faire simple, à l'heure qu'il est et au chausse pied il est peut-être possible de livrer le 20/01 mais on ne peut plus le garantir. Quelles sont les options sans trop de contraintes pour vous ? Il me semble que l'ESA avait parlé d'une indisponibilité de leur côté courant février donc en gros si on ne livre pas le 20/01 on partirait pour fin février/début mars? Si vous souhaitez qu'on en discute de vive voix lundi ou mardi, n'hésitez pas.</p>
<p>Désolé de ce mail un peu long, beaucoup d'évènements ces quelques derniers jours!</p>
<p>Bon week-end à tous.</p>
<p>Bruno</p> LFR-FSW - Task #3944: Modif de la TC_LOAD_KCOEFF : ICD + LFR FSWhttps://hephaistos.lpp.polytechnique.fr/redmine/issues/3944?journal_id=104522023-01-17T13:21:07Zbruno katra
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Closed</i></li></ul>