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

(MEAN_)PSF_TO_FIBER_SPECFLUX needs shorter FITS description #203

Open
sbailey opened this issue Nov 27, 2024 · 1 comment
Open

(MEAN_)PSF_TO_FIBER_SPECFLUX needs shorter FITS description #203

sbailey opened this issue Nov 27, 2024 · 1 comment
Assignees

Comments

@sbailey
Copy link
Contributor

sbailey commented Nov 27, 2024

Followup to #195 and #196 providing FITS-friendly shorter column descriptions:

PSF_TO_FIBER_SPECFLUX and MEAN_PSF_TO_FIBER_SPECFLUX are such long column names that they need even shorter comments to fit. The truncated versions are still understandable so this isn't urgent, but they do generate warnings that might hide other issues.

From loa/zcatalog/v1/zpix-main-backup.fits

TTYPE26 = 'PSF_TO_FIBER_SPECFLUX' / PSF light fraction within fiber (given seein
TTYPE77 = 'MEAN_PSF_TO_FIBER_SPECFLUX' / Mean PSF light fraction within 1.5arcse
@weaverba137 weaverba137 self-assigned this Dec 4, 2024
@weaverba137
Copy link
Member

I've added a fix for this into #202, please take a look. The short descriptions are:

  • PSF_TO_FIBER_SPECFLUX: 'PSF light fraction in fiber (given seeing)', 42 characters/44 available
  • MEAN_PSF_TO_FIBER_SPECFLUX: 'Mean over exp PSF light frac in fiber', 37 characters/39 available

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

2 participants