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-42 1D Redmine <p><em><font color="#505f79"> Created on 2020-11-13 14:38:44 by Didier Vibert. % Done: 100</font></em></p> <p>FittedTplMeiksinIdx is an output given for each redshift solution and refers to the igm extinction curve selected for the model. The curves comes from Meiksin 2005, 7 curves times 11 redshift bins .The index goes from 0 to 6, selecting one of the 7 curves of the redshift bin corresponding to the redshift solution.</p> <p>when the IGM extinction is not applied or when the the redshift is too small, there is no extinction applied. In such a case the output is currently left to zero.</p> <p>A non-ambiguous value of "-1" should be set instead</p> REDMINE1D-42

[RM-6123] set FittedTplMeiksinIdx output value to -1 when non applicable

Task Normal Done Done Redmine-Jira Migtation Redmine-Jira Migtation Fri, 4 Jun 2021 01:20:33 +0000 Tue, 13 Jun 2023 06:12:14 +0000 Tue, 13 Jun 2023 06:12:14 +0000 0 2 <p>Comment by Mira Sarkis on 2021-06-30 16:25:56:</p> <ul> <li>done for templatefitting.</li> <li>done for tplCombination as part of issue #6086</li> <li>already done for templatefittingLOG (since we disable igm once we determine that igm cannot be applied. IGM disabling happens pior to iterating over Meiksin values)</li> <li>Note that there is nothing to do for linemodel since it uses TF and/OR TFLOG to fit the continuum (including selecting the best IGM value)</li> </ul> <p>Comment by Mira Sarkis on 2021-07-20 13:49:19:<br/> Ajout d'un controle sur les valeurs de redshift dans TFLOG et sur les valeurs de lambda pour déterminer si la raie lya apparait ou pas.<br/> Sinon, on désactive IGM et on sort -1 comme valeur par défaut pour l'indice IGM.</p> <p>En conséquence de ce changement, pas mal d'indices IGM passe à -1 dans le it, nottament pour sp8 et qp8_qso_linemodel<br/> Sinon, suite à la désactivation de l'IGM, ie., absence de l'appel à @::ApplyMeiksinCoeff@, des légeres différences (de l'ordre de 10-6 max) apparaissent sur la pdf et certaines valeurs (proba, continuumAmp, ) de sortie de se8_galaxy_templatefitting_fft). Ceci n'implique aucun changement sur les valeurs finales des redshifts.</p> <p>Comment by Didier Vibert on 2021-07-20 15:37:03:<br/> Mira Sarkis wrote in #note-9:<br/> &gt;<br/> &gt; Sinon, suite à la désactivation de l'IGM, ie., absence de l'appel à @::ApplyMeiksinCoeff@, des légeres différences (de l'ordre de 10-6 max) apparaissent sur la pdf et certaines valeurs (proba, continuumAmp, ) de sortie de se8_galaxy_templatefitting_fft). Ceci n'implique aucun changement sur les valeurs finales des redshifts.</p> <p>ça me gène un peu: normalement une multiplication par 1.0 ou son absence ne devrait pas changer le résultat (bit-perfect) ??</p> <p>Comment by Mira Sarkis on 2021-07-20 15:48:01:<br/> Didier Vibert wrote in #note-10:<br/> &gt; Mira Sarkis wrote in #note-9:<br/> &gt; &gt;<br/> &gt; &gt; Sinon, suite à la désactivation de l'IGM, ie., absence de l'appel à @::ApplyMeiksinCoeff@, des légeres différences (de l'ordre de 10-6 max) apparaissent sur la pdf et certaines valeurs (proba, continuumAmp, ) de sortie de se8_galaxy_templatefitting_fft). Ceci n'implique aucun changement sur les valeurs finales des redshifts.<br/> &gt; <br/> &gt; ça me gène un peu: normalement une multiplication par 1.0 ou son absence ne devrait pas changer le résultat (bit-perfect) ??</p> <p>oui sauf que pour TFLOG, on a aussi changé les valeurs de zrange et donc nPaddedSamples change aussi et comme on a vu dans l'issue #5988 des légères modifications sont introduites.</p> <p>Comment by Didier Vibert on 2021-07-21 07:47:08:<br/> Mira Sarkis wrote in #note-11:<br/> &gt; Didier Vibert wrote in #note-10:<br/> &gt; &gt; Mira Sarkis wrote in #note-9:<br/> &gt; &gt; &gt;<br/> &gt; &gt; &gt; Sinon, suite à la désactivation de l'IGM, ie., absence de l'appel à @::ApplyMeiksinCoeff@, des légeres différences (de l'ordre de 10-6 max) apparaissent sur la pdf et certaines valeurs (proba, continuumAmp, ) de sortie de se8_galaxy_templatefitting_fft). Ceci n'implique aucun changement sur les valeurs finales des redshifts.<br/> &gt; &gt; <br/> &gt; &gt; ça me gène un peu: normalement une multiplication par 1.0 ou son absence ne devrait pas changer le résultat (bit-perfect) ??<br/> &gt; <br/> &gt; oui sauf que pour TFLOG, on a aussi changé les valeurs de zrange et donc nPaddedSamples change aussi et comme on a vu dans l'issue #5988 des légères modifications sont introduites.</p> <p>ah oui c'est vrai. donc c'est normal. je jette un coup d'oeil au code et je passe en feedback</p> <p>Comment by Didier Vibert on 2021-07-21 10:07:26:<br/> merge request <a href="https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/180" class="external-link" rel="nofollow">https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/180</a></p> <p>Comment by Pierre-yves Chabaud on 2021-08-10 12:39:17:<br/> Merged into @develop@ : @2060ccb@</p> Development Rank 0|zzs7g8: