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

LYA Deltas VAC Review (DR1) #32

Open
4 tasks done
dylanagreen opened this issue Dec 30, 2024 · 0 comments
Open
4 tasks done

LYA Deltas VAC Review (DR1) #32

dylanagreen opened this issue Dec 30, 2024 · 0 comments
Assignees
Labels
vac review Issues pertaining to VAC reviews

Comments

@dylanagreen
Copy link

Contact Person: Sindhu Satyavolu
Many catalog file (fits), 1 README

Initial Checks:

  • Includes README (see below)
  • Columns in ALLCAPS
  • Extension names in ALLCAPS
  • Files include units (see below)

Initial Notes:

  • I do not remember if we included units in the EDR version, i.e. degrees for RA, DEC. If we did not, then no need to add them now, if we did then please add them.
  • This looks to be an updated version of the EDR VAC, so can probably reuse the EDR README, but ported over to reference the DR1 version of the VAC. However, we will need a description of the difference between the sub directories (ciii/lya/lyb), as well as information on the scripts folder.
  • Related to the above, what is the purpose of the scripts folder, and is it necessary to release everything included therein?
@dylanagreen dylanagreen self-assigned this Dec 30, 2024
@dylanagreen dylanagreen added the vac review Issues pertaining to VAC reviews label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vac review Issues pertaining to VAC reviews
Projects
None yet
Development

No branches or pull requests

1 participant