-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: concat on axis=0 with categorical (GH10177) #10179
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
@@ -2929,6 +2929,23 @@ def test_pickle_v0_15_2(self): | |||
# | |||
self.assert_categorical_equal(cat, pd.read_pickle(pickle_path)) | |||
|
|||
def test_concat_categorical(self): | |||
df1 = pd.DataFrame(np.arange(18).reshape(6, 3), columns=["a", "b", "c"]) |
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.
add the issue number as a comment here
@jreback: I implemented your suggested changes and it seems that it doesn't break anything else, at least according to the unit tests. |
I rebased the changes against latest master. The Travis CI error is due to a timeout when creating the build environment, not sure how I can resolve it. |
pls rebase and squash. |
@@ -78,3 +78,5 @@ Bug Fixes | |||
- Bug in ``ExcelReader`` when worksheet is empty (:issue:`6403`) | |||
- Bug in ``Table.select_column`` where name is not preserved (:issue:`10392`) | |||
- Bug in ``DataFrame.interpolate`` with ``axis=1`` and ``inplace=True`` (:issue:`10395`) | |||
- Bug in `pandas.concat` with ``axis=0`` when column is of dtype category (:issue:`10177`) |
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.
use double back ticks around category
@jreback: I updated my merge request based on your feedback. |
BUG: concat on axis=0 with categorical (GH10177)
@sebp thanks! |
Contains a proposed fix for #10177