Skip to content

Downgrade rr columns to lower precision#347

Merged
sbailey merged 1 commit into
mainfrom
downgrade-rr-columns
Feb 20, 2026
Merged

Downgrade rr columns to lower precision#347
sbailey merged 1 commit into
mainfrom
downgrade-rr-columns

Conversation

@rongpu
Copy link
Copy Markdown
Contributor

@rongpu rongpu commented Feb 20, 2026

This PR makes the following data type changes for the DESI redrock catalog (redrock-*.fits) to reduce the file size. Also see desihub/desispec#2506.

Column Original Type → New Type
ZERR float64 → float32
ZWARN int64 → int32
SUBTYPE 20 characters → 3 characters (or maximum length; see below)
NPIXELS int64 → int32
NCOEFF int64 → int16
COEFF float64 → float32
CHI2 float64 → float32
DELTACHI2 float64 → float32

For SUBTYPE, if any element is more than 3 character long, the column adopts the maximum length; otherwise it is 3 characters in size.

@rongpu rongpu requested a review from sbailey February 20, 2026 05:50
@rongpu
Copy link
Copy Markdown
Contributor Author

rongpu commented Feb 20, 2026

I have tested this by running redrock on a DESI coadd file, and the resulting catalog is as expected.

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 42.179% (-0.3%) from 42.509%
when pulling a884542 on downgrade-rr-columns
into 930d936 on main.

@sbailey sbailey merged commit 3aa17a7 into main Feb 20, 2026
16 checks passed
@sbailey sbailey deleted the downgrade-rr-columns branch February 20, 2026 23:57
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

Successfully merging this pull request may close these issues.

3 participants