-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: More old issues #41697
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
TST: More old issues #41697
Conversation
mroeschke
commented
May 28, 2021
- closes Shortcut functions in transform are not grouped #19354
- closes BUG? Possible issue with multi indexing column names of different types #19517
- closes API: Inconsistent behavior with setting slices of Series indexed by MultiIndex #20414
- closes Inconsistent groupby-apply output shape and random values returned. #20420
- closes data frame group by with extra categorical column aggregates datetime64 NaT column to float NaN #20520
- closes Creating DataFrame throws: data type "bytes512" not understood #20734
- closes Addressing multiindex raises TypeError if indices that are rightmost are not present #20951
- closes BUG: Pandas groupby datetime and column then apply generates ValueError #21651
- tests added / passed
- Ensure all linting tests pass, see here for how to run them
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 @mroeschke lgtm. one suggestion.
} | ||
) | ||
df = df.set_index(["A", "B"]) | ||
with pytest.raises(KeyError, match="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.
with pytest.raises(KeyError, match="1"): | |
with pytest.raises(KeyError, match="^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.
sure, ex @simonjayhawkins comment
thanks @mroeschke |