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_purge_night should also delete future cumulative redshift directories #2388

Open
akremin opened this issue Oct 9, 2024 · 0 comments
Open

Comments

@akremin
Copy link
Member

akremin commented Oct 9, 2024

Currently, desi_purge_night is fairly straightforward and just removes night-level directories in several places. This doesn't, however, remove cumulative redshift directories from future nights that included the data being purged.

This is dangerous because the user may not realize that the purge didn't remove spectra, coadds, and redshifts that still use the data that was meant to be purged.

desi_purge_tilenight does this at the per-tile level. Consider either including some of that logic or iteratively calling that function for science tiles first.

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

No branches or pull requests

1 participant