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.

Sign up for GitHub

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

Jump to bottom

Add CARTID to the monitor.html #5

Open

andrew-saydjari opened this issue Jan 19, 2024 · 1 comment

andrew-saydjari commented Jan 19, 2024

What the title says. I could dig into qa.py more, but I would have thought that

apogee_drp/python/apogee_drp/apred/monitor.py

Line 804 in c5aaa9e

    
           headertext = '<TR bgcolor="#DCDCDC"> <TH>DATE <TH>MJD <TH># <TH>TYPE <TH>EXPTIME <TH>NREAD <TH>SHUTTER <TH>CONFIG <TH>DESIGN <TH>FIELD <TH>2D? <TH>1D? <TH>CFRAME? <TH>VISIT? <TH>2D MED FLUX\n'

was close to the right place, however, the header listed there does not match the current html pages. Am I looking at the wrong code? Trying to edit a page like https://data.sdss5.org/sas/sdsswork/data/apogee/apo/58774/58774.log.html, which are super useful.

Author

andrew-saydjari commented Jan 20, 2024

I don't see where it is written in https://github.com/sdss/apogee_drp/blob/c5aaa9e38cbd71981f51dd89a8afd2fd345bc58c/python/apogee_drp/apred/qa.py, but I do see where it is used. It would also be really helpful if the MJDexp.fits off of which this mjd.log.html were built at the sdss_id written as a column for each exposure number. I think that should be a fairly high priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment