Skip to content

DOC: core/generic/drop docs from array-like to list-like #5252

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

Merged
merged 1 commit into from
Oct 17, 2013

Conversation

jreback
Copy link
Contributor

@jreback jreback commented Oct 17, 2013

related #5248

@jreback
Copy link
Contributor Author

jreback commented Oct 17, 2013

cc @ruidc better? (I know its only 1 word.......)

jreback added a commit that referenced this pull request Oct 17, 2013
DOC: core/generic/drop docs from array-like to list-like
@jreback jreback merged commit de04269 into pandas-dev:master Oct 17, 2013
@ruidc
Copy link
Contributor

ruidc commented Oct 17, 2013

if it works, it works, but to me is not any clearer.

@jreback
Copy link
Contributor Author

jreback commented Oct 17, 2013

still unclear

can you suggests doc/code change to make more clear?

@ruidc
Copy link
Contributor

ruidc commented Oct 17, 2013

what about:
"passing a string or tuple will be treated as a single index label" ?

@jreback
Copy link
Contributor Author

jreback commented Oct 17, 2013

sure...

@sadruddin
Copy link

I thought the confusing bit of the doc is that it's not clear that one can pass a single label. So why not this:

labels: scalar or list-like

Or whatever would be appropriate to designate a single label

@ruidc
Copy link
Contributor

ruidc commented Oct 17, 2013

.... as indeed single integer labels are treated the same way, so it's not just tuples and strings that are treated this way - but I haven't delved beyond the commits today to understand when this happens

@jtratner
Copy link
Contributor

How about: single label or list-like?

@sadruddin
Copy link

that's much clearer indeed!

@ruidc
Copy link
Contributor

ruidc commented Oct 17, 2013

y

@jreback
Copy link
Contributor Author

jreback commented Oct 17, 2013

pushed via 97f5878

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants