Current dtype of photometry changes the values.

BDNYC / sedkit

Spectral energy distribution construction and analysis tools

https://sedkit.readthedocs.io/

BSD 3-Clause "New" or "Revised" License

18 stars 13 forks source link

Current dtype of photometry changes the values. #128

Closed SherelynA closed 1 week ago

SherelynA commented 1 week ago

Currently the dtype when setting photometry is np.float16, however this is not very precise and can change a magnitude of 23.93 to 23.94. We should change the dtype to np.float32.

kelle commented 1 week ago

Is there a test you could write which would fail using the current code but would pass with this change?