-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: Improve Index docstrings #36239
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
Index is now described as List instead of Set
Lgtm |
Sorry for stepping in, I think it's not accurate enough according to #36170 (comment) |
Some tests are failing... I don't recall changing anything else accept the documentation, Why is this even happening? |
The CI failures are unrelated to this change |
In my opinion a set implies that values are unique, which isn’t true for indexes. Maybe “sequence” is better? |
@topper-123 That makes sense. I also think perhaps the whole thing could be rewritten to something like "Immutable sequence used for indexing and alignment." (remove references to ndarrays, sets, or being ordered since none apply). What do you think? |
Looks more appropriate to me as well |
Rather than opening a separate PR can you make both changes here? |
Sure |
Discarded list/Set analogy. Will refer to it as Sequence
Discarded list/Set analogy. Will refer to it as Sequence.
@dsaxton I did the required changes. This should do... Right? |
Thanks @Nikhil1O1 |
Index is now described as List instead of Set
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff