search

BEAST-Fitting / megabeast

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

megabeast.py uses input file with settings #22

Closed lea-hagen closed 6 years ago

lea-hagen commented 6 years ago

Instead of the filenames and assorted settings being in megabeast.py, it reads in a file with all of that info. There's an example input file (using the METAL file structure), and documentation is updated.

lea-hagen commented 6 years ago

This also fixes the things I was puzzling about in #19.

karllark commented 6 years ago

Happy to merge this. Just checking it is working and ready for merging.

lea-hagen commented 6 years ago

Yes, I ran through the whole thing with a METAL field and it worked great. Though I'm not sure why so many of the checks are failing.

karllark commented 6 years ago

Checks are failing as we have no tests. If one actual test was added, the checks should all be successful. Something you might be willing to do?

lea-hagen commented 6 years ago

Ah, ok. I've been thinking about tests, but I'll save that for a separate branch/pull request.

karllark commented 6 years ago

Yes, definitely. I'll merge this pull request now.