Conversation

jbrockmendel

@mroeschkemroeschke added the AlgosNon-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, difflabel Jul 18, 2023
@mroeschkemroeschke requested a review from rhshadrach July 18, 2023 16:55
@@ -624,11 +624,13 @@ Other
- Bug in :meth:`DataFrame.reindex` with a ``fill_value`` that should be inferred with a :class:`ExtensionDtype` incorrectly inferring ``object`` dtype (:issue:`52586`)
- Bug in :meth:`DataFrame.shift` and :meth:`Series.shift` and :meth:`DataFrameGroupBy.shift` when passing both "freq" and "fill_value" silently ignoring "fill_value" instead of raising ``ValueError`` (:issue:`53832`)
- Bug in :meth:`DataFrame.shift` with ``axis=1`` on a :class:`DataFrame` with a single :class:`ExtensionDtype` column giving incorrect results (:issue:`53832`)
- Bug in :meth:`Index.sort_values` whena ``key`` is passed (:issue:`52764`)
Copy link
Member

Choose a reason for hiding this comment

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

Nit: when a

Copy link
Member

Choose a reason for hiding this comment

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

Also - move to the Indexing section?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch, will update.

As to the section, its pretty subjective but I usually think of "indexing" as about Series/DataFrame/loc.iloc getitem/setitem methods

Copy link
Member Author

Choose a reason for hiding this comment

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

updated+green

Choose a reason for hiding this comment

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

lgtm

@rhshadrachrhshadrach added Bug IndexRelated to the Index class or subclasseslabels Jul 21, 2023
@rhshadrachrhshadrach added this to the 2.1 milestone Jul 21, 2023
@rhshadrachrhshadrach merged commit 2e881c3 into pandas-dev:main Jul 21, 2023
@jbrockmendeljbrockmendel deleted the bug-index-sort_values-key branch July 21, 2023 15:13
Sign up for free to join this conversation on . Already have an account? Sign in to comment
AlgosNon-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, diffIndexRelated to the Index class or subclasses
None yet

Successfully merging this pull request may close these issues.

BUG: Index.sort_values with key