LPP » INSTRU » Solar Orbiter LFR » LFR-FSW

Possible qu'il faille ajuster la valeur du paramètre delta_f0_2 du waveform picker (se référer à la doc du VHDL pour la signification des paramètres de réglage des snapshots). La valeur actuelle date d'il y a longtemps (avant la 3.0.0.22) et le VHDL a évolué depuis. Demander à Alexis ce qu'il en pense.

void set_wfp_delta_f0_f0_2( void )
{
    unsigned int delta_snapshot;
    unsigned int nb_samples_per_snapshot;
    float delta_f0_in_float;

    delta_snapshot = waveform_picker_regs->delta_snapshot;
    nb_samples_per_snapshot = (parameter_dump_packet.sy_lfr_n_swf_l[0] * CONST_256) + parameter_dump_packet.sy_lfr_n_swf_l[1];
    delta_f0_in_float = (nb_samples_per_snapshot / 2.) * ( (1. / FREQ_F2) - (1. / FREQ_F0) ) * FREQ_F2;

    waveform_picker_regs->delta_f0      = delta_snapshot - floor( delta_f0_in_float );
    waveform_picker_regs->delta_f0_2    = DFLT_WFP_DELTA_F0_2;
}

avec

#define DFLT_WFP_DELTA_F0_2         0x30    // 48 = 11 0000, max 7 bits

Petite précision aussi sur les graphes de Thomas : pour moi seul les écarts à F1 sont préoccupants. Pour F0 et F2, cela reste la caractérisation déjà obervée et utilisée pour faire la RFD (qui a été acceptée à priori).

Je viens de faire un diff sur mes fichiers wf_handler.c entre 3.0.0.22 et la révision locale. J'ai vu un gros bug, qui est apparu à l'époque où j'ai dû enlever toutes les constantes littérales du code pour satisfaire les demandes du CNES.

J'avais en local:

deltaT_F0 = DELTAT_F0;
    deltaT_F1 = DELTAF_F1;
    deltaT_F2 = DELTAF_F2;

Au lieu de:

deltaT_F0 = DELTAT_F0;
    deltaT_F1 = DELTAT_F1;
    deltaT_F2 = DELTAT_F2;

Je joins une version 3.2.0.12 modifiée pour valider ça. Normalement, ça devrait aller!

paul leroy wrote:

Je viens de faire un diff sur mes fichiers wf_handler.c entre 3.0.0.22 et la révision locale. J'ai vu un gros bug, qui est apparu à l'époque où j'ai dû enlever toutes les constantes littérales du code pour satisfaire les demandes du CNES.

J'avais en local:
[...]

Au lieu de:
[...]

Je joins une version 3.2.0.12 modifiée pour valider ça. Normalement, ça devrait aller!

AH ! C'est une bonne piste ça !
Je teste immédiatement

Tu ne m'avais pas réattribué l'issue petit scarabée.

Je préfèrerais attendre la livraison finale du filtre pour la 3.2.0.13, ça pose un problème? Je pense qu'Alexis doit me transmettre des nouveaux coefficients.

Détail: si l'issue est "Closed", elle n'apparaît plus dans le tableau de bord alors il faut faire gaffe à ne pas y poser de question.

paul leroy wrote:

Tu ne m'avais pas réattribué l'issue petit scarabée.

Je préfèrerais attendre la livraison finale du filtre pour la 3.2.0.13, ça pose un problème? Je pense qu'Alexis doit me transmettre des nouveaux coefficients.

Je ne suis pas sûr que tu auras les coeff avant la livraison, j'ai causé avec Alexis et on livrera peut-être tel quel avec mise à jour des coeff pour les calibrations au CNES.
Aussi le pb avec la 3.2.0.12 c'est que j'en ai 4 versions (compilée par Alexis avec et sans -mfix-b2bst + 2 de toi). J'ai du coup beaucoup de tests avec ce numéro dans les HK alors que le soft diffère, ça commence à être limite pour la traçabilité.
Si vraiment c'est pas grand chose, je préfèrerai continuer avec une 3.2.0.13.
Merci d'avance