-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: GH2072 add example and warning on fallback indexing for float indexes. #3128
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
can you make the doc lines wrap, no more than 80 char width. (I get flagged for this all the time!) Looks good otherwise @y-p ? |
also, maybe add an example of using the 'float' selection, e.g say A is your previous indexer column
|
very bad idea is not the best pharsing IMO. explain the problem. explain the caveats, Also, I can't parse: I haven't followed this issue, so i'm looking at the doc with fresh eyes... ;) |
Oh, re 80 char limit on documentation - stop the PEP8 insanity! it's not even code! |
Good point @y-p. How about this? |
looks good to me (and now you are taking style points from @y-p )....! |
Hey- if you don't want pedantry, don't ask for my opinion on docs. |
and if you wouldn't mind, pls rebase into 1 commit
change the pick to fixup (or squash if you want) for all but the first then
|
@jreback - done. I wondered how that worked in git - thanks. |
this looks great I'll leave open for a couple of days in case you want to change anything |
yep, this is good. |
thanks for the pr! DOC: GH2072 add example and warning on fallback indexing for float indexes.
@adamgreenhall I realized that the last part of your example is correct, but not illustrating what I think you want.
Same here
Doing something like this might make the example more clear (then you can use
|
#2072 (comment)