-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
Fix doc #59793
Fix doc #59793
Conversation
fbourgey
commented
Sep 13, 2024
- closes DOC: Intro to data structures #59789
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.
Thanks for the PR!
doc/source/user_guide/dsintro.rst
Outdated
the length of **index**. If the index is not provided, it defaults | ||
to ``RangeIndex (0, 1, 2, …, n)``. |
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.
n
here is not defined, and I think in this notation would be 0
, but this is not the notation used to construct a RangeIndex. In particular, RangeIndex(0)
here is wrong. Can you make this RangeIndex(1)
.
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.
I made the change.
/preview |
Website preview of this PR available at: https://pandas.pydata.org/preview/pandas-dev/pandas/59793/ |
doc/source/user_guide/dsintro.rst
Outdated
provided. The value will be repeated to match the length of **index**. | ||
If ``data`` is a scalar value, the value will be repeated to match | ||
the length of **index**. If the **index** is not provided, it defaults | ||
to ``RangeIndex (1)``. |
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.
Can you remove the space so it is RangeIndex(1)
.
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.
done
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
Thanks @fbourgey! |