Skip to content

Conversation

jbrockmendel
Copy link
Member

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. import change.

@@ -77,6 +77,7 @@
ABCMultiIndex,
ABCSeries,
)
from pandas.core.dtypes.inference import is_hashable
Copy link
Contributor

Choose a reason for hiding this comment

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

should be part of the: pandas.core.dtypes.common import above

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed+green

@jreback jreback added Bug Categorical Categorical Data Type Indexing Related to indexing on series/frames, not to indexes themselves labels Jul 8, 2019
@jreback jreback added this to the 0.25.0 milestone Jul 8, 2019
@jreback jreback merged commit fc86f8d into pandas-dev:master Jul 9, 2019
@jreback
Copy link
Contributor

jreback commented Jul 9, 2019

thanks @jbrockmendel liking your pushing small orthogonal patches.

@jbrockmendel jbrockmendel deleted the catin branch July 10, 2019 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Categorical Categorical Data Type Indexing Related to indexing on series/frames, not to indexes themselves
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pd.Categorical.contains([item, otheritem]) raises ValueError
2 participants