astropy / astropy-tutorials

Tutorials for the Astropy Project
BSD 3-Clause "New" or "Revised" License
288 stars 170 forks source link

New tutorial to calculate radial velocity from multiple measurements #550

Open alexbinks opened 2 years ago

alexbinks commented 2 years ago

This tutorial is designed to assess the multiplicity status of a star and calculate a "final" radial velocity for stars when there are several (> 2) individual measurements. Whilst intended for RV measurements, the method can be applied for any parameter that needs to be weighted from different sources. Principally the program: 1) reads in a table of RV measurements for any set of stars 2) makes a decision on whether the star is multiple or likely single 3) calculates a final RV from the individual measurements.

review-notebook-app[bot] commented 2 years ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

hamogu commented 2 years ago

Thanks for contributing! This looks interesting, but I think you might want to see if the indexing logic can be simplified a bit. At least I don't understand it - but maybe that's just because I don't know that much about the topic.

kelle commented 2 years ago

This looks really promising! Thanks so much for the contribution and we'll work on giving it a robust review ASAP. I'm also going to start a thread in the #learn channel on slack.