Comment by Didier Vibert on 2023-12-13 18:42:37:
Bon, une explication possible.
Les SNR provenant du fit Gaussien en mode ratio ne sont pas très exploitables. En effet, ils sont basés sur l´incertitude de la mesure d'une amplitude unique (celle du modèle avec toutes les raies fixées), qui est assez contrainte et donc précise. Le SNR est donc trompeur. Si les ratios étaient justes, ce SNR serait bon. Il faudrait avoir des variances sur les ratios pour retrouver quelque chose de plus plausible.
En attendant le SNR directIntegration est correct: 2.2

Comment by Vincent Le Brun on 2023-12-14 07:51:24:
du coup je suggère de ne pas les calculer et encore moins de les afficher. Cela dit certains flux ont l'air très faux aussi (cf #8544)

Comment by Didier Vibert on 2023-12-15 10:57:00:
Après vérif, les SNR (en mode template ratio) de toutes les raies sont bien les mêmes (ils ne sont pas scalés par les amplitudes des ratios, car ce scaling intervient au numérateur et dénominateur du SNR, ie l'amplitude et son incertitude sont scalés par le ratio). Si on run en extended results, les ratios amplitude/amplitude_uncertainties sont bien les mêmes et correspondent au ratio amplitude_globale/amplitude_globale_uncertainty.
D'ailleurs cette dernière quantité n'est pas sortie, je rajoute du code pour ça.
En revanche les snr en flux ne donnent pas la même chose, car la largeur est dépendante de la longueur d'onde, même avec une LSF constante (sigma constant) à cause du terme dispersion de vitesse qui est constant en vitesse et donne donc des largeurs en longueur d'onde dépendantes de la longueur d'onde.

Ensuite à voir si on veut conserver ces quantités individuelles par raies dans le cas des templates-ratio, vu qu'elles n'ont pas grand sens... en revanche le SNR venant du fit global de toutes les raies a un sens.

Comment by Vincent Le Brun on 2023-12-15 11:25:23:
je suis partisan de ne pas calculer ces valeurs individuelles vu qu'elles n'ont pas de sens physique

Comment by Didier Vibert on 2023-12-19 16:36:25:

• J'ai supprimé des output (mis à NAN) les incertitudes des flux par raie dans le cas tplratio.
• J'ai ajouté l'incertitude et le SNR de l'amplitude globale du tplratio
• J'ai ajouté le SNR par raie dans les cas rules ou free (il y avait déjà le flux et l'incertitude), mais ça évite de faire le calcul et permet des filtrages avec vizu

Comment by Didier Vibert on 2023-12-21 12:15:13:
MR: https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/571

Comment by Didier Vibert on 2024-01-26 10:32:41:
some small modifications suggested by @aallaoui

Comment by Pierre-yves Chabaud on 2024-01-31 16:04:11:
Merged into @develop@ (@c84e9cf8@)