desihub / desihub/desida

Peculiar Velocity (PV) VAC Review (Post-DR1)

Open
#60 13 comments 0 reactions 1 assignee Claimed by @weaverba137 View on GitHub
vac review
Dominant language
Python
Stars
7
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Contact Person: Anthony Carr, @AnthonyC958

7 catalog files (csv, dat), 5407 random catalog files (fits, hdf5), 1 README

Initial Checks:
- [X] Includes README
- [x] Columns in ALLCAPS
- [x] Extension names in ALLCAPS
- [x] Files include units
- [x] VAC publication passed CWR

Initial Notes:

- Is it possible to convert the *.dat files to *.csv? It appears they only need commas?
- Would it be significant additional work to convert the CSV files to FITS? That could potentially reduce the size of the catalogs, and units could be added to the columns.
- Based on an initial sampling, the FITS files in the `mocks/` directory do not
* have column names in ALLCAPS
* have extension names
* include units

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.