[REDMINE1D-260] [RM-8178] [Refacto] Cleanup snr calculations Created: 12/Jul/23  Updated: 22/Sep/23  Resolved: 22/Sep/23

Status: Done
Project: 1D Redmine
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: Redmine-Jira Migtation Assignee: Redmine-Jira Migtation
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2023-07-11 14:48:29 by Fanny Dufresne. % Done: 100

  • The same formula snr = dtm/sqrt(mtm) is used in severeal places and redefined several times -> find the right place to create a method to be used by all
    ex:
  • tempaltefittingsolve.cpp (buildExtremaResults, FitSNR calc)
  • linemodel.cpp (precomputecontinuumfit)
  • Method getFitContinuum_snr() has been defined but unused -> which need ? to delete ?

MR: https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/532

 Comments   
Comment by Redmine-Jira Migtation [ 22/Sep/23 ]

Comment by Didier Vibert on 2023-07-17 14:03:57:
yep !

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.

Effectivement @CContinuumManager::getFitContinuum_snr()@ est à supprimer.
Le commentaire:
<pre><code class="cpp">
// This SNR estimate maybe needs to use observed spectrum with lines removed ?
</code></pre>

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.

Comment by Redmine-Jira Migtation [ 22/Sep/23 ]

Comment by Didier Vibert on 2023-08-31 07:59:46:
question de @fdufresne:
> 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 ?

effectivement c'est en doublon. Il faut je pense supprimer FittedTplSNR. Et se contenter d'utiliser partout TExtremaResult.fittedTpl.SNR

Comment by Redmine-Jira Migtation [ 22/Sep/23 ]

Comment by Pierre-yves Chabaud on 2023-09-15 13:52:14:
Merged into @develop@ (@0d2180ee@)

Generated at Sat Feb 10 15:30:55 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.