-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pr47 #25
Pr47 #25
Conversation
* @param rebx pointer to the REBOUNDx extras instance. | ||
* @param Nvalues Length of times and values arrays (must be equal for both). | ||
* @param times Array of times at which the corresponding values are supplied. | ||
* @param times Array of values at each corresponding time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
param name should be values
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor typo in rebound.h
header comment for rebx_create_interpolator()
that I'll fix after I merge into my branch. Otherwise, everything looks great. I closed both document related issues on your branch and mine.
Pr47 Former-commit-id: 0cbba9d
Hey Stanley,
I've updated a bit of the discussion in ParameterInterpolation.ipynb and have added a header to interpolation.c so that it shows up in the documentation. If you could take a look at those and merge this in your branch together with any changes you want to make, we can merge this into master!
I've also taken the opportunity to make some updates to the documentation that had fallen behind. Thank you SO much for all your great work on this!