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

desi_proc_dashboard not showing all nights #1915

Closed
sbailey opened this issue Nov 21, 2022 · 4 comments
Closed

desi_proc_dashboard not showing all nights #1915

sbailey opened this issue Nov 21, 2022 · 4 comments
Assignees
Projects

Comments

@sbailey
Copy link
Contributor

sbailey commented Nov 21, 2022

running desi_proc_dashboard on Himalayas starts out with

Searching /global/cfs/cdirs/desi/spectro/redux/himalayas for: ['20220613' '20220612' ... '20201212' '20201209']

however the final html dashboard only includes months 202105 through 202206, i.e. the SV nights aren't included.

Perhaps related: the original main survey processing was on Cori, while the SV nights were processed on Perlmutter, though in this case I'm running desi_proc_dashboard on Perlmutter and it is only showing the Cori nights. Or perhaps it is unrelated to Cori vs. Perlmutter and some other filter is going on.

@sbailey sbailey added this to To do in Himalayas via automation Nov 21, 2022
@sbailey
Copy link
Contributor Author

sbailey commented Dec 8, 2022

This might be a conflict with a cronjob not including SV nights stepping on the results of running it by hand. Check that first on cori21 crontab.

@sbailey
Copy link
Contributor Author

sbailey commented Dec 8, 2022

I think I've seen a similar phenomenon with desi_zproc_dashboard. Check that too. i.e. get to a state where both the dashboard and zdashboard for himalayas includes all nights from all months covered in the production.

@akremin
Copy link
Member

akremin commented Dec 8, 2022

I turned off the cron job on cori21 and find that running desi_proc_dashboard on perlmutter does produce an html file with outputs for the months I request. I did not verify every individual night is present, but there are no missing months as mentioned in the issue.

For desi_proc_zdashboard I do see many missing months. My guess is that it has to do with the redshift jobs not being present in the processing tables. The SV months are shown, which were the ones run on perlmutter with redshifts as part of the standard pipeline. The main survey were run outside the main pipeline with desi_tile_redshifts, which don't add rows to the processing table. I will look into that.

@sbailey
Copy link
Contributor Author

sbailey commented Jan 11, 2023

I believe all cases are understood. For desi_proc_dashboard it was a cronjob with a limited daterange overwriting the by-hand invocation outputs; for desi_proc_zdashboard it was due to redshift jobs submitted with desi_tile_redshifts which doesn't update the processing table (see #1952 for the workaround in desi_run_night so that we don't need to use desi_tile_redshifts for typical processing).

Although desi_zproc_dashboard could be updated to be based upon files on disk without looking at the processing table, that is a bit messier to know what is supposed to exist. I'm going to close this ticket and we can revive this in a separate focused ticket if needed.

@sbailey sbailey closed this as completed Jan 11, 2023
Himalayas automation moved this from To do to Done Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants