[PIPE1D-53] [0.24.0] Mask values no properly used Created: 09/Sep/21 Updated: 18/Oct/21 Resolved: 18/Oct/21 |
|
| Status: | Done |
| Project: | DRP 1D pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Normal |
| Reporter: | vlebrun | Assignee: | Ali Allaoui |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Description |
|
The DRP-1D should use only 0 values of the mask provided by the DRP2D |
| Comments |
| Comment by vlebrun [ 17/Sep/21 ] |
|
We have investigated further, and it happens that the 1D Pipeline actually uses the mask values provided by the conversion parser in the pipeline. We have actually discovered three issues from the file provided by Kiyoto Yabe in
These problems explain the bad redshift measurement, and mask values have to be corrected
|
| Comment by Kiyoto Yabe [ 17/Sep/21 ] |
|
Thank you for the investigation. What about this one (pfsObject-00001-00000-0,0-0000000000000318-006-0x7ace412dc05971ff.fits |
| Comment by vlebrun [ 17/Sep/21 ] |
|
Indeed the values are different for the extension #2 (#8 seems to be identical), did you try DRP-1D on this one? we'll try our version on our side
The mask seems to be correct with respect to strong features around 1220-1230 nm |
| Comment by vlebrun [ 17/Sep/21 ] |
|
and now we measure the correct redshift, which shows that everything seems OK. On the other side, the mask in the resampled data table are still weird. Should we raise a PIPE-2D ticket ? |
| Comment by Kiyoto Yabe [ 17/Sep/21 ] |
|
Yes, the recent comparison (check_redshift_determination.pdf |
| Comment by Kiyoto Yabe [ 17/Sep/21 ] |
|
Also here I invite price for discussion on mask values in pfsObject. |
| Comment by vlebrun [ 17/Sep/21 ] |
|
the value of the std deviation of 0.2 indicates that there are still a large number of catastrophic failures. Let's check that ... |
| Comment by Kiyoto Yabe [ 20/Sep/21 ] |
|
The latest weekly products (at IPMU) are here: https://pfs.ipmu.jp/internal/devarch/ipmu/pipe_e2e/ I also added reference redshift and [OII] luminosity in csv file: https://pfs.ipmu.jp/internal/devarch/ipmu/pipe_e2e/check_redshift_determination.csv Original input spectra are not in the weekly products but I can send you specific objects if you need them. |
| Comment by price [ 22/Sep/21 ] |
|
I have recently discovered and fixed some bugs in the generation of the mask. I hope these fixes will make it to a weekly release soon. |
| Comment by Kiyoto Yabe [ 06/Oct/21 ] |
|
With the latest run (2D=w.2021.40 and 1D=0.24.0), the comparison between input and output redshifts is like this: check_redshift_determination_20211006.pdf The result looks obviously strange. Could you check this in your side? pfsObjects are here: Input redshifts are here: https://pfs.ipmu.jp/internal/devarch/ipmu/pipe_e2e/check_redshift_determination.csv
|
| Comment by vlebrun [ 06/Oct/21 ] |
|
For the spectrum I looked, the noise level does not make sense; it is about 10^12 while the spectrum has a (normal) 10^16 average value, hence the noise weighted fit cannot work properly. On this plot Weekly_run_noise_level.png |
| Comment by Kiyoto Yabe [ 18/Oct/21 ] |
|
The weekly tests in this morning showed that the comparisons https://pfs.ipmu.jp/internal/devarch/ipmu/pipe_e2e/check_redshift_determination.pdf https://pfs.ipmu.jp/internal/devarch/ipmu/pipe_e2e_extended/check_redshift_determination.pdf were fair enough. So I think we can close this ticket and post any further issues in different ticket (maybe PIPE1D-38?). |