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.

Sign up for GitHub

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

Jump to bottom

Exposures table produced by Progress.get_exposures() does not assign units to RA, Dec. #73

Closed
weaverba137 opened this issue Nov 8, 2017 · 2 comments
Closed

Exposures table produced by Progress.get_exposures() does not assign units to RA, Dec. #73

weaverba137 opened this issue Nov 8, 2017 · 2 comments
Assignees
@weaverba137
Labels
missing feature
Milestone
DC2017 Prep

Comments

@weaverba137
Copy link
Member

Based on a recently-produced exposures.fits file, generated by Progress.get_exposures(), the RA, Dec columns do not have units assigned to them at all. Obviously the units should be degrees. This may be inherited from the progress.fits file, which also does not assign units for RA, Dec.
@weaverba137 weaverba137 added this to the DC2017 Prep milestone Nov 8, 2017
@weaverba137 weaverba137 self-assigned this Nov 8, 2017
@weaverba137
Copy link
Member Author

And it looks like this problem goes all the way back to the desi-tiles.fits file in desimodel, which does not assign units for RA, Dec.

@weaverba137
Copy link
Member Author

The exposures.fits file produced by Progress.get_exposures() now does set units on RA, Dec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@weaverba137 weaverba137

Labels
missing feature
Projects
None yet
Milestone
DC2017 Prep
Development

No branches or pull requests
1 participant
@weaverba137