desihub / desisurveyops

Scripts and code for DESI Survey Operations
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Tiles with ~large sky residuals that need to be marked bad or better processed #4

Closed schlafly closed 1 year ago

schlafly commented 2 years ago

These are currently "unsure" and typically have lots of significant redshifts at z=1.8. Often have SKYCHI2PDF warning. May just need to have their component exposures marked bad so that NTS will reobserve them.

Affected: 20140, 21321, 23276, 24633, 24634, 26147

e.g. https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/20140/20211117/tile-qa-20140-thru20211117.png

sframe files show lots of sky amplitude variation from fiber to fiber that is either a gradient or cloudy? https://data.desi.lbl.gov/desi/users/raichoor/main-status/sframesky/sframesky-20211117-00109233.png

https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/20140/20211117/tile-qa-20140-thru20211117.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/21321/20220114/tile-qa-21321-thru20220114.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/23276/20211117/tile-qa-23276-thru20211117.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/24633/20211219/tile-qa-24633-thru20211219.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/24634/20211219/tile-qa-24634-thru20211219.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/26147/20211219/tile-qa-26147-thru20211219.png

julienguy commented 2 years ago

Would be interesting to display the sky residual brightness as a function of focal plane coordinates.

schlafly commented 2 years ago

No idea if this is the right plot, but here's something: image I took all the sky fibers, and extracted the median flux from the frame-r files (i.e., not sframe, cframe, coadd ...) with 6000 < lam < 7000, and then plotted. So, yes, obvious structures on sub-petal scales, I think? If I were better, I should be adding the skies to the sframes and doing this, and then might be taking advantage of the fiberflat?

julienguy commented 2 years ago

gradient due to moon light or twilight?

schlafly commented 2 years ago

Here's a gallery of these cases: https://data.desi.lbl.gov/desi/users/schlafly/sky/largeskyresid.pdf It's likely I don't yet have the direction to the moon right, etc.; I need to check this more. But the most striking thing to me about these is that the trend is more radial than a gradient.

I suspect this means that I'm primarily looking at the fiber flat field. I didn't want to work with the sframe files because I wanted the sky. I don't immediately see where the fiberflat lives, so maybe the best choice is to use the sframe + sky files, which will be flattened?

I'd also be interested in understanding the THRPUTCORR extension of the sky models; every fiber gets an empirical sky amplitude fit, maybe based on bright sky lines? That would be news to me?

sybenzvi commented 2 years ago

Several more exposures with similar SKYCHI2PDF warnings:

araichoor commented 2 years ago

And another one (I m adding the png file, as links may become obselete once/if we re-run the pipeline):

geordie666 commented 2 years ago

A few more of these. Back-to-back exposures (139344 and 139345) on 20220611 had multiple sky CHI2PDF alerts:

https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/25124/20220611/tile-qa-25124-thru20220611.png https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/25109/20220611/tile-qa-25109-thru20220611.png

See (closed) issue #59 for details.

schlafly commented 2 years ago

@akremin, can we reprocess the following tiles now that the new sky subtraction code is in? We'd then want to re-QA them and see if we can mark them as good and archive & mtl-update.

This list merges a few different issues together to hopefully get all the tiles that I think may be able to be solved by improved sky subtraction.

TILEID LASTNIGHT   QA  
int64    int64    str6 
------ --------- ------
  2295  20220211 unsure
  5805  20220505 unsure
  7009  20211215 unsure
  8801  20220219 unsure
 20140  20211117 unsure
 21321  20220114 unsure
 21809  20220512 unsure
 21828  20220612 unsure
 21829  20220516 unsure
 21984  20220514 unsure
 23276  20211117 unsure
 24633  20211219 unsure
 24634  20211219 unsure
 24896  20220516 unsure
 24905  20220516 unsure
 24919  20220516 unsure
 25109  20220611 unsure
 25124  20220611 unsure
 26147  20211219 unsure
akremin commented 2 years ago

I have removed and re-run all of the exposures associated with the above tiles on the corresponding nights. None of the tiles were observed on other nights and none of them were archived, so this was a clean procedure. The redshifts are now done and the tsnr afterburner + night QA have been rerun on the new outputs. The scripts used for this cleanup are in /global/cfs/cdirs/desi/spectro/redux/daily/run/scripts/custom/large_sky_residuals_20220909.

These tiles are now ready for re-QA.

schlafly commented 2 years ago

Great! Here are the results:

  2295  20220211 unsure
  5805  20220505 good
  7009  20211215 good
  8801  20220219 good
 20140  20211117 unsure
 21321  20220114 unsure
 21809  20220512 good
 21828  20220612 unsure
 21829  20220516 good
 21984  20220514 good
 23276  20211117 unsure
 24633  20211219 unsure
 24634  20211219 good
 24896  20220516 good
 24905  20220516 good
 24919  20220516 good
 25109  20220611 good
 25124  20220611 unsure
 26147  20211219 good

So 2/3 of these are now survey quality and marked good. Let's discuss briefly on the data call later today, but I'm going to recommend we mark the others bad and reprocess. Some examples: dark tile 2295: image The line at z=1.8 is related to sky residuals. But it is very subtle. I could be convinced to keep it as good, I guess.

20140: image Still pretty bad.

25124: image More arguable, but I've left it unsure.

21828: image More arguable, but I've left it unsure.

I don't see any easy approaches to improving the sky subtraction further at the desispec level. One could imagine next steps being fitting the sky simultaneously among the three cameras (e.g., to avoid jumps at camera boundaries), and introducing additional terms into redrock to soak up sky residuals. But those would be more major operations.

schlafly commented 2 years ago

@akremin, following the discussion on yesterday's survey-ops call, let's go ahead and mark

  2295  20220211 unsure
 20140  20211117 unsure
 21321  20220114 unsure
 21828  20220612 unsure
 23276  20211117 unsure
 24633  20211219 unsure
 25124  20220611 unsure

as bad and reprocess. Then I need to clear their QA status to none.

akremin commented 1 year ago

The above tiles have all had the corresponding night's exposures removed, along with redshifts. None had later night's redshifts. I have rerun the tsnr afterburner for all nights.

Back to you, @schlafly .

schlafly commented 1 year ago

Thanks, this is in. Closing!