-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Crosstab Preserves Dtypes release note #24789
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
Comments
I'm not aware of an intentional change making it obsolete, so I guess it counts as a regression. That said, the behavior change in this note was a side-effect of #22019, not at all part of the intention. It looks like the edits that #22019 made in tests.frame.test_axis_select_reindex and tests.reshape.test_pivot are no longer necessary. I'd say revert those, remove the note, and call it a win. |
agreed. I'll put up a PR quick. |
At the time, pandas-dev#22019 seemed to break API in a few places. Since then, those API breaking changes have disappeared, so the changes to the tests and the release note are no longer necessary. Closes pandas-dev#24789
At the time, pandas-dev#22019 seemed to break API in a few places. Since then, those API breaking changes have disappeared, so the changes to the tests and the release note are no longer necessary. Closes pandas-dev#24789
At the time, pandas-dev#22019 seemed to break API in a few places. Since then, those API breaking changes have disappeared, so the changes to the tests and the release note are no longer necessary. Closes pandas-dev#24789
The note at http://pandas.pydata.org/pandas-docs/version/0.24.0rc1/whatsnew/v0.24.0.html#crosstab-preserves-dtypes seems incorrect (it's a code block instead of an ipython block, but after making that change and running it).
On master, I get the same output as 0.23.4
0.23.4
master
@jbrockmendel this came from #22019. Do you know if the note is obsolete, or has there been a regression?
The text was updated successfully, but these errors were encountered: