desihub / desispec

DESI spectral pipeline
BSD 3-Clause "New" or "Revised" License
36 stars 24 forks source link

QL not catching varying CCD amp bias levels #705

Open sbailey opened 6 years ago

sbailey commented 6 years ago

Testing http://desi-1.kpno.noao.edu:9090 using code from desispec #702 (now tag 0.25.0) and qlf #253, mostly looking at Observing History window and drill-downs.

From the QLF trend plots, it appears that the CCD amp offset (bias) and noise values are jumping around, which appears to be a bug in the input simulations. See desihub/desisim#443. However, quicklook should have caught this as an error -- if the real data were jumping around this much we'd want to know.

Srheft commented 6 years ago

@sbailey great to know. Thank you very much!

rkehoe commented 6 years ago

@sbailey Can you indicate if response from @srheft has resolved this issue? It sounds like buggy sims may be part of what you were observing. Definitely specifications on metrics need to be tightened and tuned. If that is the purpose of this ticket, then we should leave it open. Otherwise, looking to identify issues that are still out there. Thanks

Srheft commented 6 years ago

@rkehoe : thanks for the followup. This one needs multiple runs on the current set of exposures to catch the outliers to the noise and bias trend. Like I sad above, I'll have to set up a full night to catch the CCD stat patterns.