Skip to content

more informative error message with np.min or np.max on unordered Categorical #34053

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

Merged
merged 7 commits into from
May 19, 2020

Conversation

simonjayhawkins
Copy link
Member

@simonjayhawkins simonjayhawkins added Categorical Categorical Data Type Error Reporting Incorrect or improved errors from pandas labels May 7, 2020
@@ -778,6 +778,7 @@ Other
- Bug in :meth:`DataFrame.__dir__` caused a segfault when using unicode surrogates in a column name (:issue:`25509`)
- Bug in :meth:`DataFrame.plot.scatter` caused an error when plotting variable marker sizes (:issue:`32904`)
- :class:`IntegerArray` now implements the ``sum`` operation (:issue:`33172`)
- more informative error message with ``np.min`` or ``np.max`` on unordered Categorical (:issue:`33115`)
Copy link
Member

Choose a reason for hiding this comment

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

Capitalize "more", ":class:`Categorical`"

Copy link
Member Author

Choose a reason for hiding this comment

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

done

Copy link
Contributor

@topper-123 topper-123 left a comment

Choose a reason for hiding this comment

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

LGTM (when the tests pass).

@simonjayhawkins
Copy link
Member Author

I think all comments addressed. will merge tomorrow if no further comments.

@simonjayhawkins simonjayhawkins added this to the 1.1 milestone May 19, 2020
@simonjayhawkins simonjayhawkins merged commit 21a10d1 into pandas-dev:master May 19, 2020
@simonjayhawkins simonjayhawkins deleted the message-regression branch May 19, 2020 08:45
simonjayhawkins added a commit to simonjayhawkins/pandas that referenced this pull request May 19, 2020
…message with np.min or np.max on unordered Categorical)
simonjayhawkins added a commit that referenced this pull request May 19, 2020
…th np.min or np.max on unordered Categorical) (#34246)
@simonjayhawkins simonjayhawkins modified the milestones: 1.1, 1.0.4 May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Categorical Categorical Data Type Error Reporting Incorrect or improved errors from pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

REGR: unhelpful error message with np.min on unordered Categorical
4 participants