Skip to content

Conversation

jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Nov 26, 2020

The main thing being changed here is CategoricalIndex._concat to better match the behavior of all the other FooIndex._concat methods. Everything else being changed here is just cleanup that this makes possible.

cc @jorisvandenbossche

@jreback jreback added Categorical Categorical Data Type Reshaping Concat, Merge/Join, Stack/Unstack, Explode labels Nov 26, 2020
Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jreback jreback added this to the 1.2 milestone Nov 26, 2020
Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need a whatsnew note as this is a behavior change (now a non-matching category is coerced to object)

@jbrockmendel
Copy link
Member Author

whatsnew added + greenish (test_area_lim)

@jreback jreback merged commit 22007d3 into pandas-dev:master Nov 29, 2020
@jbrockmendel jbrockmendel deleted the ref-to_safe_for_reshape branch November 29, 2020 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Categorical Categorical Data Type Reshaping Concat, Merge/Join, Stack/Unstack, Explode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API: Index.append behaviour with categoricals
2 participants