https://pfspipe.ipmu.jp/jira This file is an XML representation of an issue en-us 8.3.4 803005 13-09-2019 https://pfspipe.ipmu.jp/jira/browse/REDMINE1D-319 1D Redmine <p><em><font color="#505f79"> Created on 2023-09-21 08:07:40 by Ali Allaoui. % Done: 100</font></em></p> <p>Review CLineModelElement::m_OutsideLambdaRange and CLineModelElement::m_LineIsActiveOnSupport definition (maybe move them to CLineModelElementList) : CLineModelElement::prepareSupport</p> <p>CLineModelElementList should be a vector of CLlineModelElement not shared_ptr&lt;CLlineModelElement&gt;</p> REDMINE1D-319

[RM-8312] [multiobs][linemodel] Adapt AbstractFitter::fitAmplitude

Task Normal Done Done Redmine-Jira Migtation Redmine-Jira Migtation Thu, 21 Sep 2023 18:16:28 +0000 Tue, 6 Feb 2024 18:40:02 +0000 Tue, 6 Feb 2024 18:40:02 +0000 0 1 <p>Comment by Ali Allaoui on 2023-09-29 13:31:01:<br/> <a href="https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/553" class="external-link" rel="nofollow">https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/553</a></p> <p>Comment by Ali Allaoui on 2023-10-02 12:50:21:<br/> <a href="https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60" class="external-link" rel="nofollow">https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60</a></p> <p>Comment by Didier Vibert on 2023-10-17 08:49:50:<br/> un truc à vérifier aussi: dans @TLineModelResult::updateFromModel@ il y a des calculs qui sont faits via des appels à </p> <ul> <li>@CLineModelFitting::getCumulSNRStrongEL()@</li> <li>@CLineModelFitting::getLinesAboveSNR()@</li> <li>@CLineModelFitting::getElementList().GetModelNonZeroElementsNDdl()@</li> <li>@CLineModelFitting::getOutsideLinesSTD(...)@</li> </ul> <p>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</p> <p>Comment by Ali Allaoui on 2023-10-17 08:58:53:<br/> oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait<br/> 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</p> <p>Comment by Didier Vibert on 2023-10-17 10:04:15:<br/> Ali Allaoui wrote in #note-5:<br/> &gt; oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait<br/> ok</p> <p>&gt; 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<br/> 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))</p> <p>Comment by Ali Allaoui on 2023-10-17 10:11:24:<br/> Didier Vibert wrote in #note-6:<br/> &gt; Ali Allaoui wrote in #note-5:<br/> &gt; &gt; oui j'avais repéré déjà, GetModelNonZeroElementsNDdl c'est fait<br/> &gt; ok<br/> &gt; <br/> &gt; &gt; 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<br/> &gt; 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))</p> <p>Ok donc déplacer getLinesAboveSNR dans CLineModelFitting, je verrais si je gagne à conserver des bouts dans CSpectrumModel</p> <p>Comment by Ali Allaoui on 2023-11-15 08:54:17:<br/> Plus de bug en mono obs, une revue peut être intéressante pendant que je track les bugs multiobs.<br/> Il y a aussi une issue dataset-parameters qui modifie se8_multiobs pour le rendre comparable à se8<br/> <a href="https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60" class="external-link" rel="nofollow">https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/60</a></p> <p>Comment by Pierre-yves Chabaud on 2024-01-17 16:38:42:<br/> Merged into @develop@ (@0210a55e@)</p> Development Rank 0|zzszof: