Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Read noise errors sometimes triggered due to CTE #425

Open
djschlegel opened this issue Jul 27, 2024 · 5 comments
Open

Read noise errors sometimes triggered due to CTE #425

djschlegel opened this issue Jul 27, 2024 · 5 comments
Assignees
Labels
documentation Updates to documentation (in readthedocs or the wiki).

Comments

@djschlegel
Copy link

The CTE problem can trigger read noise errors in NightWatch. Recent examples from the night 07-23-2024 had two backup tiles that triggered this error on the z1-D amplifiers (exposures 245014, 245021). I suspect the read noise is computed in the overscan region, and is thrown off by the CTE when there is a bright object in fibers near the overscan. (Is this interpretation correct?)

Options?

(1) Do nothing except in documentation. This is triggering off a real problem (CTE), even if the error is not obvious. This could be added to the NightWatch documentation. (One other comment is that most of the documentation is on the "Troubleshooting" page linked off NightWatch, rather than the "Documentation" page.)

(2) Run the CTE correction code within NightWatch to correct for the CTE before measuring the read noise.

(3) Explicitly trigger on the CTE.

@djschlegel
Copy link
Author

Screenshot 2024-07-25 at 7 29 19 AM

@sybenzvi
Copy link
Contributor

@djschlegel, I'd like to try option 2 (run the CTE correction code in Nightwatch) if that is possible. I'm ignorant of where in the processing the correction is happening so this could be easier said than done. We can discuss at the next survey-ops call.

I have the impression this is not a common error -- in the night you highlighted, there are only two exposures affected unless I missed something. But, I will add some documentation to the Nightwatch troubleshooting wiki page.

@sybenzvi sybenzvi added the documentation Updates to documentation (in readthedocs or the wiki). label Jul 29, 2024
@sybenzvi sybenzvi self-assigned this Jul 29, 2024
@schlafly
Copy link

The CTE correction code relies on calibration parameters that are fit from the analysis of the flat field exposures. We would have to bring that processing over to nightwatch in order to use it. Alternatively, we could do something like use the CTE parameters from the last good night of data from the offline pipeline, which sounds easier to me but would require some extra bookkeeping. David, would you mind elaborating on what the major features in your plot are? Where is the overscan region? I'm not accustomed to looking at these raw (?) files.

@sbailey
Copy link
Collaborator

sbailey commented Aug 12, 2024

@schlafly can you also clarify whether applying the CTE corrections require the full Horne 1986 row-by-row optimal extractions, or whether that is just for the measurement of the CTE parameters? I'm concerned that their could be a substantial processing time cost to including CTE corrections in Nightwatch.

@schlafly
Copy link

Yes, the spectra are extracted a handful of times using the full row-by-row optimal extractions. It probably takes O(1 minute) / CCD in total.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Updates to documentation (in readthedocs or the wiki).
Projects
None yet
Development

No branches or pull requests

4 participants