ericagol / NbodyGradient.jl

N-body integrator computes derivatives with respect to initial conditions for TTVs, RV, Photodynamics & more
MIT License
20 stars 9 forks source link

Error in order of drifts and kicks #13

Closed ericagol closed 5 years ago

ericagol commented 5 years ago

In the case that the integrator uses kicks instead of drifts, the drift should come before the kick, while right now the reverse is true:

https://github.com/ericagol/NbodyGradient/blob/ef6c69fe696f302977d0d317a9c16c5910494352/src/ttv.jl#L1518-L1522

ericagol commented 5 years ago

This order is the order used in the Dehnen & Hernandez 2017 equation (40) integrator.