search

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

Compatibility with new version of NbodyGradient #35

Closed ericagol closed 3 years ago

ericagol commented 3 years ago

@langfzac Is this Jupyter notebook consistent with your newest version of NbodyGradient?

https://github.com/ericagol/NbodyGradient/blob/master/examples/ttv_example.ipynb

langfzac commented 3 years ago

No, but the one I just pushed to my fork is. https://github.com/langfzac/NbodyGradient/blob/master/examples/ttv_example.ipynb

bmlindor commented 3 years ago

Just a quick Q, How does this compare with the compensated_summation branch?

On Jan 26, 2021, at 8:26 PM, Zach Langford notifications@github.com wrote:

No, but the one I just pushed to my fork is.https://github.com/langfzac/NbodyGradient/blob/master/examples/ttv_example.ipynb https://github.com/langfzac/NbodyGradient/blob/master/examples/ttv_example.ipynb — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ericagol/NbodyGradient/issues/35#issuecomment-768019455, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEW6SCL75AAHE7U23KZOYTLS36IWDANCNFSM4WUPP4NQ.

ericagol commented 3 years ago

@bmlindor I believe that branch has been merged with the master branch in the main repository. @langfzac has been developing some new code to interface with the integrator, including structures to hold the variables, new ways to initiate the initial conditions, and integration of more general hiearchies. This version he is developing on his fork of this repository.

langfzac commented 3 years ago

Notebook updated in #36