Skip to content

API: Change default for Index.union sort #25007

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.

Sign up for GitHub

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

Closed
wants to merge 14 commits into from
Closed

API: Change default for Index.union sort #25007

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
aac172c
[WIP]: API: Change default for Index.union sort
TomAugspurger Jan 29, 2019
d4bcc55
update test
TomAugspurger Jan 29, 2019
45c827c
fixups
TomAugspurger Jan 30, 2019
68b72a6
Merge remote-tracking branch 'upstream/master' into 24959-union-default
TomAugspurger Jan 30, 2019
8716f97
multi
TomAugspurger Jan 30, 2019
f7056d5
doc typo
TomAugspurger Jan 30, 2019
e82cbb1
intersection
TomAugspurger Jan 30, 2019
2a2de25
whatsnew
TomAugspurger Jan 30, 2019
5c3da74
update whatsnew
TomAugspurger Jan 30, 2019
ce6d1db
Merge remote-tracking branch 'upstream/master' into 24959-union-default
TomAugspurger Jan 31, 2019
52a2f2f
symdiff
TomAugspurger Jan 31, 2019
bb848f1
whatsnew
TomAugspurger Jan 31, 2019
b15dc7e
doc
TomAugspurger Jan 31, 2019
27b5b16
index multi
TomAugspurger Jan 31, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
whatsnew
  • Loading branch information
@TomAugspurger
TomAugspurger committed Jan 31, 2019
commit bb848f19a68ccb3a3a74f366a26dd152d9f4ad12
5 changes: 4 additions & 1 deletion doc/source/whatsnew/v0.24.1.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ API Changes
Changing the ``sort`` parameter for :meth:`Index.union`
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The default ``sort`` value for :meth:`Index.union` has changed from ``True`` to ``None``.
The default ``sort`` value for :meth:`Index.union` has changed from ``True`` to ``None`` (:issue:`24959`).
The default *behavior* remains the same: The result is sorted, unless

1. ``self`` and ``other`` are identical
Expand Down Expand Up @@ -53,6 +53,9 @@ This allows ``sort=True`` to now mean "always sort". A ``TypeError`` is raised i

idx.union(idx, sort=True)

The same change applies to :meth:`Index.symmetric_difference`, which would previously not
sort the result when ``sort=True`` but the values could not be compared.

Changed the behavior of :meth:`Index.intersection` with ``sort=True``
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Expand Down