Skip to content

Fix the unit information on the output#268

Merged
ayllon merged 1 commit intodevelopfrom
fix/units
Jul 10, 2020
Merged

Fix the unit information on the output#268
ayllon merged 1 commit intodevelopfrom
fix/units

Conversation

@ayllon
Copy link
Member

@ayllon ayllon commented Jul 7, 2020

The [] are not needed when going through CCfits (maybe it is for cfitsio?).
They end as-is on the output catalog, which confuses other FITS implementations like Astropy, which fails to identify the unit.
i.e

In [5]: t['world_centroid_alpha'].unit                                                                                                                                                        
Out[5]: UnrecognizedUnit([deg])

After this fix

In [12]: t['world_centroid_alpha'].unit                                                                                                                                                       
Out[12]: Unit("deg")

@ayllon ayllon requested review from marcschefer and mkuemmel July 7, 2020 08:27
@ayllon ayllon merged commit f1a9bdd into develop Jul 10, 2020
@ayllon ayllon deleted the fix/units branch July 10, 2020 07:06
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.

2 participants