-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: DataFrame.pivot not respecting None as column name #48293
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
+1 for this. |
Will open a pr to backport to 1.5rc then, will convert this to draft till the other pr is merged then to simplify the backport |
# Conflicts: # pandas/core/frame.py
cc @mroeschke I think this is ready for another look now |
# Conflicts: # doc/source/whatsnew/v1.6.0.rst
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.
One small item otherwise LGTM
Co-authored-by: Matthew Roeschke <[email protected]>
Thanks @phofl |
…8293) * BUG: DataFrame.pivot not respecting None as column name * Add gh ref * Fix test * Add comment * Update doc/source/whatsnew/v1.6.0.rst Co-authored-by: Matthew Roeschke <[email protected]> Co-authored-by: Matthew Roeschke <[email protected]>
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Columns can never be empty when called by the user, should we deprecate positional arguments here and then remove the default? avoids the check on our side and simplifies things a bit