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-71 1D Redmine <p><em><font color="#505f79"> Created on 2020-06-03 08:08:23 by Didier Vibert. % Done: 100</font></em></p> <p>lorsqu'on décide de conserver plus d'un continu par ordre de merit pour fitter le linemodel (paramètre @linemodelsolve.linemodel.continuumfit.count@ &gt; 1) </p> <p>le best chi2 et paramètres du linemodel correspondant retenu à chaque z, dans @CLineModelElementList::fit@, correspondent à ceux du dernier continu testé (donc a priori le moins bon...), au lieu de comparer pour regarder lequel donne le meilleur résultat (cette comparaison est faite dans le cas du tpl-ratio) </p> <p>cf source:RedshiftLibrary/src/lib/linemodel/elementlist.cpp@master#L2569 où la comparaison est faite pour le cas tpl-ratio, entre les différents templates mais aussi les différents continus </p> REDMINE1D-71

[RM-5806] linmodel free/rules: erreur avec plusieurs continus

Task Normal Done Done Redmine-Jira Migtation Redmine-Jira Migtation Fri, 4 Jun 2021 01:21:48 +0000 Wed, 5 Jul 2023 07:48:15 +0000 Wed, 5 Jul 2023 07:48:15 +0000 0 1 <p>Comment by Didier Vibert on 2020-06-03 08:40:43:<br/> La question est dans le ticket #5875.</p> <p>Comment by Mira Sarkis on 2022-07-01 16:36:21:<br/> J'ai profité pour faire un minor refactoring de CLineModelFitting::fit et puis de remplacer "tplshape" par "tplratio" .</p> <p>mr-lib : <a href="https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/362" class="external-link" rel="nofollow">https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/362</a><br/> mr-datase-params: <a href="https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/30" class="external-link" rel="nofollow">https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/30</a></p> <p>Comment by Mira Sarkis on 2022-07-04 15:26:37:<br/> Les ITs sont valides, par rapport aux résultats de fix_issue_6961.</p> <p>Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants: </p> <ul> <li>se8_roman_prism</li> <li>se8_roman_grism,<br/> qui semble ne pas etre testés/validés dans le cadre 6961! </li> </ul> <p>Note: j'ai réussi à réduire le nombre de ligne de @fit@, en passant de 871 à 671 lignes!</p> <p>Comment by Didier Vibert on 2022-07-13 14:30:46:<br/> Mira Sarkis wrote in #note-14:<br/> &gt; Les ITs sont valides, par rapport aux résultats de fix_issue_6961.<br/> &gt; <br/> &gt; Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants: <br/> &gt; * se8_roman_prism<br/> &gt; * se8_roman_grism,<br/> &gt; qui semble ne pas etre testés/validés dans le cadre 6961! <br/> &gt;</p> <p>normalement les it roman passent maintenant sans changements dans 6961. Tu peux confirmer et verifier que c'est aussi le cas pour cette issue. merci</p> <p>Comment by Mira Sarkis on 2022-07-13 16:32:06:<br/> Didier Vibert wrote in #note-17:<br/> &gt; Mira Sarkis wrote in #note-14:<br/> &gt; &gt; Les ITs sont valides, par rapport aux résultats de fix_issue_6961.<br/> &gt; &gt; <br/> &gt; &gt; Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants: <br/> &gt; &gt; * se8_roman_prism<br/> &gt; &gt; * se8_roman_grism,<br/> &gt; &gt; qui semble ne pas etre testés/validés dans le cadre 6961! <br/> &gt; &gt;<br/> &gt; <br/> &gt; normalement les it roman passent maintenant sans changements dans 6961. Tu peux confirmer et verifier que c'est aussi le cas pour cette issue. merci</p> <p>Oui les IT romans passent maintenant.</p> <p>Par contre, les variations @lm_continuumComponent_nocontinuum@ présente des différences maintenant!</p> <p>Comment by Mira Sarkis on 2022-07-18 12:56:56:<br/> lib: merged in @fix_issue_6652@ : @cbd7b9c3@<br/> dataset-params: merged in @fix_issue_6652@ : @031aa0f7@</p> <p>Note: les ITs présentent des false-positive diff, correspondant au changement du la valeur d'initialisation du tplRatioName qui change de "None" à "undefined". </p> <p>Comment by Pierre-yves Chabaud on 2022-08-03 08:01:09:<br/> pylibamazed: merged into @develop@ : @4e408666@<br/> dataset-parameters: merged into @develop@ : @8342207@</p> Development Rank 0|zzs7mo: