-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Fix rename for MultiIndex columns DataFrame #4004
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
Fix rename for MultiIndex columns DataFrame #4004
Conversation
can u hook up to Travis? also pls add a release notes mention |
Okay I think I've hooked it up to Travis. Do I have to run manual build? |
push another commit or git commit --amend -C HEAD (will reset the latest coit hash) then git push yourfork yourbranch -f then Travis will build it |
Should I add the release note mention here? doc/source/v0.11.1.txt |
release notes in doc/source/release.rst |
ok...looks fine, except need to rebase, don't want to have those
and remove the offending commit (just delete it) 05bd8618ab21701f7067969e84c9200c88ed4848 also prob want to then
will rewrite this branch |
Fix rename for MultiIndex columns DataFrame
Sorry I'm still learning git, I hope I did your last instruction right, but I'm lost with deleting the commit Merge branch. |
Thanks for the fix, I ran into the same problem today and was about to start on a fix but you beat me to it. :) I have tested your code and it works as expected for me, matching the functionality for indexes already put in by 3165. Let me know if there is anything I can do to help. |
@ogiaquino it looks like you rebased successfully, good work! |
…umns_DataFrame Fix rename for MultiIndex columns DataFrame
DataFrame rename doesn't rename a MultiIndex column and it flattens the columns. Closes #3165 which is a fix for index but not columns.