phoebe-project / phoebe2

PHOEBE - Eclipsing Binary Star Modeling Software
http://phoebe-project.org
GNU General Public License v3.0
80 stars 30 forks source link

Fixed two bugs with JKTEBOP backend RVs. #701

Closed jsinkbaek closed 1 year ago

jsinkbaek commented 1 year ago
  1. To calculate RV semi-amplitude (K), system semi-major-axis was used previously instead of component sma (sma vs. center of mass). Before, any change in the mass ratio had no effect on the modelled rvs.

  2. Small left-over discrepancy corrected, as RV amplitude K did not take eccentricity into account. Now includes term 1/(sqrt(1-ecc^2)).

Closes #699