search

BEAST-Fitting / megabeast

Hierarchical Bayesian Model for Ensembles of Dust Extinguished Stellar Populations
http://megabeast.readthedocs.io/
2 stars 11 forks source link

Assumed file name may result in an error when generating naive maps #69

Open petiay opened 2 years ago

petiay commented 2 years ago

(This is minor). Although the standard BEAST outputs are stored in a stats.fits file, referring to stats.fits in this line of make_naive_maps.py may result in an error if one has changed the name of the output file where stats.fits does not exist in a file name. (Actually the error I encountered ends up overwriting the very stats.fits file! ) For this reason I'd lobby for only referring to .fits instead of to stats.fits, unless values_per_pixel.hd5 without a preceding stats is an expected input somewhere else at this point and it's hard to change that.

christinawlindberg commented 1 year ago

I've run into this same issue, where, because my stats files had a suffix, the code just ends up overwriting my original file.