Comment by Ali Allaoui on 2023-09-29 13:31:01:
https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/553

Comment by Ali Allaoui on 2023-10-02 12:50:21:
https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60

Comment by Didier Vibert on 2023-10-17 08:49:50:
un truc à vérifier aussi: dans @TLineModelResult::updateFromModel@ il y a des calculs qui sont faits via des appels à

• @CLineModelFitting::getCumulSNRStrongEL()@
• @CLineModelFitting::getLinesAboveSNR()@
• @CLineModelFitting::getElementList().GetModelNonZeroElementsNDdl()@
• @CLineModelFitting::getOutsideLinesSTD(...)@

or toutes ces méthodes font un calcul sur un model ou spectre, donc une observation seulement. Il faut donc modifier ces méthodes pour mutualiser (accumuler) le résultat sur toutes les obs

Comment by Ali Allaoui on 2023-10-17 08:58:53:
oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait
getLinesAboveSNR je pensais le laisser dans SpectrumModel et donc utiliser le snr calculé sur l'observation et non sur toutes les observations, pour ensuite aggréger toutes les lines above snr

Comment by Didier Vibert on 2023-10-17 10:04:15:
Ali Allaoui wrote in #note-5:
> oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait
ok

> getLinesAboveSNR je pensais le laisser dans SpectrumModel et donc utiliser le snr calculé sur l'observation et non sur toutes les observations, pour ensuite aggréger toutes les lines above snr
c'est pas forcément le mieux, lorsque une raie est visible sur plusieurs obs à la fois, si les obs se chevauchent., dans ce cas là il faut calculer le SNR sur la totalité des pixels pour lequel la raie est visible, car les SNR utilisés sont des SNR "direct integration" et non pas des SNR paramétriques: fitted_amplitude / sqrt(var(fitted_amplitude))

Comment by Ali Allaoui on 2023-10-17 10:11:24:
Didier Vibert wrote in #note-6:
> Ali Allaoui wrote in #note-5:
> > oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait
> ok
>
> > getLinesAboveSNR je pensais le laisser dans SpectrumModel et donc utiliser le snr calculé sur l'observation et non sur toutes les observations, pour ensuite aggréger toutes les lines above snr
> c'est pas forcément le mieux, lorsque une raie est visible sur plusieurs obs à la fois, si les obs se chevauchent., dans ce cas là il faut calculer le SNR sur la totalité des pixels pour lequel la raie est visible, car les SNR utilisés sont des SNR "direct integration" et non pas des SNR paramétriques: fitted_amplitude / sqrt(var(fitted_amplitude))

Ok donc déplacer getLinesAboveSNR dans CLineModelFitting, je verrais si je gagne à conserver des bouts dans CSpectrumModel

Comment by Ali Allaoui on 2023-11-15 08:54:17:
Plus de bug en mono obs, une revue peut être intéressante pendant que je track les bugs multiobs.
Il y a aussi une issue dataset-parameters qui modifie se8_multiobs pour le rendre comparable à se8
https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60

Comment by Pierre-yves Chabaud on 2024-01-17 16:38:42:
Merged into @develop@ (@0210a55e@)