search

BASTAcode / BASTA

BASTA: The BAyesian STellar Algorithm
https://basta.readthedocs.io/
MIT License
21 stars 3 forks source link

Make it easier to make a star.xml for frequencies #43

Closed amaliestokholm closed 9 months ago

amaliestokholm commented 9 months ago

This pull request makes it easier to make a star.fre file into an xml file.

Instead of relying on the given order of colunns (and boolean flags), this reads in the header and makes choices based on the available information. If n is not given, it 'computes' it.

Furthermore, this fixes that chi2_astero does not always give 4 output arguments, as assummed by bastamain.

MLWinther commented 9 months ago

Nice change, can you update the example in the documentation (just the simplest working case, I'm working on a general methods section) to reflect this change? And good catch on the chi2_astero output, that's my bad.

amaliestokholm commented 9 months ago

I have updated the documentation. I also added a few input checks on the error columns - feel free to suggest other checks as well.