-
Notifications
You must be signed in to change notification settings - Fork 385
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
Error in analysis due to pandas bug #30
Comments
ghost
changed the title
Error in Part 2 flights analysis example/graph due to a long-standing pandas bug
Error in analysis due to pandas bug
Jun 12, 2019
See pandas-dev/pandas#22509 (comment) for comparison of |
ping @TomAugspurger, have you seen this yet? you're so responsive on pandas, but this has been here for a few days, just wondering. |
Seen it. Won't have time to look for a while though.
…On Thu, Jun 13, 2019 at 2:39 PM pilkibun ***@***.***> wrote:
ping @TomAugspurger <https://github.com/TomAugspurger>, have you seen
this yet? you're so responsive on pandas, but this has been here for a few
days, just wondering.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#30?email_source=notifications&email_token=AAKAOIWCBZQLGBFEKHER52LP2KPAXA5CNFSM4HWFCC22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXU2COI#issuecomment-501850425>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKAOISS4ILJW5ELFMTGZ23P2KPAXANCNFSM4HWFCC2Q>
.
|
ghost
mentioned this issue
Jul 15, 2019
ghost
closed this as completed
Jul 29, 2019
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@TomAugspurger, I found pandas-dev/pandas#22509 by chasing down an error in part 2. You use the broken form of
groupby().transform('rank')
in "Chaining methods", to create this graph:If you use
dep_time.rank()
instead you get the correct resultThe text was updated successfully, but these errors were encountered: