[REDMINE1D-215] [RM-7529] add warning when 2nd pass window size is too small Created: 05/Jul/23  Updated: 28/Jul/23  Resolved: 28/Jul/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

Attachments: PNG File clipboard-202304051455-dskq6.png    

 Description   

Created on 2022-09-28 09:03:07 by Didier Vibert. % Done: 100

add warning when 2nd pass window size is too small

this can be checked, a posteriori, when the dz has been estimated: we should check that z_estim +/- dz is inside the window, if not, add a dedicated warning

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



 Comments   
Comment by Redmine-Jira Migtation [ 28/Jul/23 ]

Comment by Didier Vibert on 2023-04-05 12:57:45:
pour info, sur ce sujet, il faut faire qq chose: en effet, lorsque la fenêtre 2nd passe est trop étroite et que la précision en z est plus large on arrive à ça, où on voit que la fenêtre -3sigma, +3simga sur laquelle on intègre la proba, déborde de la fenêtre seconde passe.

(se8_roman_prism IT best galaxy candidate)

Comment by Redmine-Jira Migtation [ 28/Jul/23 ]

Comment by Fanny Dufresne on 2023-05-15 15:35:48:
Du coup est-ce que ce qu'on veut vérifier c'est que z_estim +/- *3 ** sigma est dans la fenêtre? (Plus large que juste z_estim +/- sigma)

Comment by Redmine-Jira Migtation [ 28/Jul/23 ]

Comment by Didier Vibert on 2023-05-16 16:07:49:
Fanny Dufresne wrote in #note-7:
> Du coup est-ce que ce qu'on veut vérifier c'est que z_estim +/- *3 ** sigma est dans la fenêtre? (Plus large que juste z_estim +/- sigma)

oui, ça me parait bien de prendre +/- 3 sigma

@fdufresne tu peux rajouter un lien vers la Merge request dans un commentaire de l'issue stp (ça évite d'aller le rechercher dès qu'on regarde l'issue) merci

Comment by Redmine-Jira Migtation [ 28/Jul/23 ]

Comment by Didier Vibert on 2023-05-16 16:09:46:
Didier Vibert wrote in #note-9:
> Fanny Dufresne wrote in #note-7:
> > Du coup est-ce que ce qu'on veut vérifier c'est que z_estim +/- *3 ** sigma est dans la fenêtre? (Plus large que juste z_estim +/- sigma)
>
> oui, ça me parait bien de prendre +/- 3 sigma
>
> @fdufresne tu peux rajouter un lien vers la Merge request dans un commentaire de l'issue stp (ça évite d'aller le rechercher dès qu'on regarde l'issue) merci

ah j'avais pas vu tu l'as mis dans la description !

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