spacetelescope / specviz

An interactive astronomical 1D spectra analysis tool.
http://specviz.readthedocs.io
BSD 3-Clause "New" or "Revised" License
43 stars 31 forks source link

How to handle flux unit electrons/s #136

Closed pllim closed 8 years ago

pllim commented 8 years ago

Some data has

BUNIT = 'ELECTRONS/S'

but Astropy does not recognize it. specviz needs to auto-magically correct that somehow.

nmearl commented 8 years ago

Can we define some special equivalencies or something, and add them to some type of astropy unit conversion registry?

pllim commented 8 years ago

That might work but would be overkill. I was thinking just adding an if-else hack in loaders.py within _flux_unit_from_header() function. Is that alright or too hacky?

nmearl commented 8 years ago

No, no, that's perfectly fine!