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

Add continuum check to arc lamp standards #330

Open
sybenzvi opened this issue Jan 19, 2023 · 3 comments
Open

Add continuum check to arc lamp standards #330

sybenzvi opened this issue Jan 19, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@sybenzvi
Copy link
Contributor

The easiest way to check for stray light is elevated continuum (particularly in the r and z cameras) during arc cals. Probably this can be done by cutting out all prominent lines and looking at the resulting integrated flux in each camera.

@sybenzvi sybenzvi added the enhancement New feature or request label Jan 19, 2023
@sbailey
Copy link
Collaborator

sbailey commented Dec 17, 2023

@jose-bermejo @sybenzvi pinging you about this Nightwatch enhancement request. There was a faint light left on the past few nights (20231215 and 20231216) and this test might have been able to catch it.

@deisenstein
Copy link

My Jupyter code is here on NERSC:

/global/u1/e/eisenste/public/CalibrationLamps.ipynb

What this does is compare between two sets of cals. Ideally, one holds one set "fixed" at some version thought to be good. I'm sure one could make different choices about how to separate the lines from the continuum, and surely write better python.

I note that the lamps have changed over time, such that I have had to occasionally change the reference night.

@sybenzvi
Copy link
Contributor Author

Thanks @deisenstein , we based the calibration standards now in Nightwatch off your notebook. There is a continuum check for the LEDs and an EW check for the arc lines, but we didn't implement the median-filtered plots of the arc lamps, which is useful for picking out stray light. We also correlated the drifts in the light levels to the air temperature in the dome but didn't implement automatic adjustments in the standards yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants