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-260 1D Redmine <p><em><font color="#505f79"> Created on 2023-07-11 14:48:29 by Fanny Dufresne. % Done: 100</font></em></p> <ul class="alternate" type="square"> <li>The same formula snr = dtm/sqrt(mtm) is used in severeal places and redefined several times -&gt; find the right place to create a method to be used by all<br/> ex:</li> <li>tempaltefittingsolve.cpp (buildExtremaResults, FitSNR calc)</li> <li>linemodel.cpp (precomputecontinuumfit)</li> </ul> <ul class="alternate" type="square"> <li>Method getFitContinuum_snr() has been defined but unused -&gt; which need ? to delete ?</li> </ul> <p>MR: <a href="https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/532" class="external-link" rel="nofollow">https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/532</a></p> REDMINE1D-260

[RM-8178] [Refacto] Cleanup snr calculations

Task Normal Done Done Redmine-Jira Migtation Redmine-Jira Migtation Tue, 11 Jul 2023 18:15:22 +0000 Thu, 21 Sep 2023 18:32:40 +0000 Thu, 21 Sep 2023 18:32:40 +0000 0 1 <p>Comment by Didier Vibert on 2023-07-17 14:03:57:<br/> yep !</p> <p>Le plus simple il me semble sera d'utiliser le membre @CTemplateFittingResult::SNR@. Il n'est utilisé que par @COperatorTplCombinatiion@ dont le calcul est un peu plus compliqué (combinaison linéraire de template au lieu d'un seul). Il faudrait implémenter le calcul de ce membre dans la méthode @CTemplateFittingResult::set_at_redshift@ et ensuite récupérer cette valeur partout ailleurs.</p> <p>Effectivement @CContinuumManager::getFitContinuum_snr()@ est à supprimer. <br/> Le commentaire: <br/> &lt;pre&gt;&lt;code class="cpp"&gt;<br/> // This SNR estimate maybe needs to use observed spectrum with lines removed ?<br/> &lt;/code&gt;&lt;/pre&gt;</p> <p>est à oublier, lors de l'utilisation de l'operateur templatefitting dans le cadre du linemodel on utilise des templates orthogonaux au linemodel (ou en ayant mis à zéros les supports des raies), et donc le SNR en tient compte.</p> <p>Comment by Didier Vibert on 2023-08-31 07:59:46:<br/> question de @fdufresne: <br/> &gt; Dans TExtremaResult on a à la fois un champ FittedTplSNR et un champ fittedTpl.SNR. Est-ce un doublon involontaire ou bien est-ce que ça a un objectif en particulier ?</p> <p>effectivement c'est en doublon. Il faut je pense supprimer FittedTplSNR. Et se contenter d'utiliser partout TExtremaResult.fittedTpl.SNR</p> <p>Comment by Pierre-yves Chabaud on 2023-09-15 13:52:14:<br/> Merged into @develop@ (@0d2180ee@)</p> Development Rank 0|zzsyo7: