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

Misleading column names in measurements.arrow #603

Open
marxide opened this issue Jan 14, 2022 · 0 comments
Open

Misleading column names in measurements.arrow #603

marxide opened this issue Jan 14, 2022 · 0 comments
Labels
low priority Issue is not of immediate concern.

Comments

@marxide
Copy link
Contributor

marxide commented Jan 14, 2022

The measurements file contains three columns that I find have misleading names: psf_bmaj, psf_bmin, and psf_pa. The values stored in these columns are the sizes of the Gaussian components after deconvolving with the PSF. The current names make it seem like they are storing the size of the PSF. They should be renamed to e.g. bmaj_deconv, etc. Note that the documentation accurately describes these columns, it's just the names that I find misleading.

@marxide marxide added the low priority Issue is not of immediate concern. label Jan 14, 2022
@github-actions github-actions bot added this to To do in Pipeline Backlog Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Issue is not of immediate concern.
Projects
Development

No branches or pull requests

1 participant