-
Notifications
You must be signed in to change notification settings - Fork 215
[DOC] Typo correction in getting started guide #1842
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
Conversation
Thank you for contributing to
|
I made a contribution to a different issue on the same branch, but I removed that commit so this PR should only involve yesterday's contributions of the two typos in the getting start guide. |
Hey ! Thanks for contributing, I think there is a confusion on the |
Hi @baraline, |
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.
LGTM after the changes
It is good to fill out the initial PR as best you can. You can delete template bits if they do not apply. Good practice to link to issue if you can, i.e. "Fixes #1841" which will close that issue on merge. |
Reference Issues/PRs
Fixes #1841.
Does your contribution introduce a new dependency? If yes, which one?
No additional dependencies.
Any other comments?
The decision for removing the redundant 'n_timepoints' was supported by Antoine.