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-91 1D Redmine <p><em><font color="#505f79"> Created on 2023-06-06 08:53:44 by Mira Sarkis. % Done: 0</font></em></p> <p>@m_OutsideLambdaRangeOverlapThreshold@ is hardcoded to 33% which is too small.<br/> We need to make sure that the line center (mu) belongs to lambdarange, so at least 70% of overlapThreshold is required.</p> <p>Todo: remove the hardcoded value in @CLineModelElement@ and add a new corresponding parameter.</p> <p>Check run: /net/CESAM/amazed/msarkis/GEWG_50K_2020_8102/output_0.42_linemeasFitGaussien with lbdaRange.max = 12500.<br/> Bug could be observed for spectraID = 00246009 where z_linemeas = 0.909784 --&gt; lbda_Ha_obs = 12536.97A &gt;12500A<br/> !<span class="image-wrap" style=""><img src="https://pfspipe.ipmu.jp/jira/secure/attachment/17002/17002_246009+_output_0.42_linemeasFitGaussien.png" style="border: 0px solid black" /></span>!<br/> Note that the computed gaussian Flux is very high, ie., 3.23*10^-12</p> <p> Other spectra with similar problem: 00001517, 00009358, 00018571, 00030199,00054383</p> REDMINE1D-91

[RM-8113] [linemeas] Review minimum OverlapThreshold (33%) for lines outside lambdarange

Task Normal Open Unresolved Redmine-Jira Migtation Redmine-Jira Migtation Tue, 13 Jun 2023 04:58:39 +0000 Thu, 11 Jan 2024 18:24:31 +0000 0 2 <p>Comment by Didier Vibert on 2023-06-15 10:03:31:<br/> TODO:</p> <ul> <li>create a new linemodel parameter for the overlap threshold</li> <li>output the overlap % (in pixels) for each measured lines (need to extract some information about the sampling resolution of the input spectrum =&gt; improve @CSpectrumSpectralAxis::GetResolution@ and improve this part of the code: <a href="https://gitlab.lam.fr/CPF/cpf-redshift/-/blob/release-0.42/RedshiftLibrary/src/lib/linemodel/element.cpp#L461" class="external-link" rel="nofollow">https://gitlab.lam.fr/CPF/cpf-redshift/-/blob/release-0.42/RedshiftLibrary/src/lib/linemodel/element.cpp#L461</a>)</li> <li>set in the default parameters a value of 50% mini</li> </ul> <p>Comment by Didier Vibert on 2024-01-11 13:50:33:<br/> @vlebrun a priori cette issue conflicte avec le multi-obs. On la post-pone après ? même si elle devra sans doute être traitée rapidement pour Euclid ? </p> <p>Comment by Vincent Le Brun on 2024-01-11 13:53:44:<br/> Didier Vibert wrote in #note-8:<br/> &gt; @vlebrun a priori cette issue conflicte avec le multi-obs. On la post-pone après ? même si elle devra sans doute être traitée rapidement pour Euclid ?</p> <p>ben oui. Et comme il semblerait que la couverture spectrale de NISP soit un peu plus grande que prévue, on a un peu de marge si le range en redshift n'est pas modifié</p> <p>Comment by Didier Vibert on 2024-01-11 13:56:17:<br/> Vincent Le Brun wrote in #note-9:<br/> &gt; Didier Vibert wrote in #note-8:<br/> &gt; &gt; @vlebrun a priori cette issue conflicte avec le multi-obs. On la post-pone après ? même si elle devra sans doute être traitée rapidement pour Euclid ?<br/> &gt; <br/> &gt; ben oui. Et comme il semblerait que la couverture spectrale de NISP soit un peu plus grande que prévue, on a un peu de marge si le range en redshift n'est pas modifié</p> <p>ok pour le range en z, cette issue devrait aussi prendre en compte les trous dans la couverture en longueurs d'onde (cf #7982)</p> Development Rank 0|zzsxdj: