Skip to content

BUG: merging an Integer EA rasises #23262

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

Merged
merged 15 commits into from
Dec 19, 2018
+58 −8
Merged

BUG: merging an Integer EA rasises #23262

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
1e0539d
fix bug #GH23020
makbigc Oct 21, 2018
9310361
change after review
makbigc Nov 4, 2018
cb2d4d1
2nd change after review
makbigc Nov 11, 2018
42477c3
3rd change after review
makbigc Nov 17, 2018
6a9e360
Separate the test and add more cases
makbigc Nov 18, 2018
373582e
add more types into the test_merge_on_int_array
makbigc Dec 5, 2018
f817cad
isort pandas/core/reshape/merge.py
makbigc Dec 9, 2018
8140ee1
Add _values_for_factorize in period.py
makbigc Dec 15, 2018
c425b24
Remove is_period_dtype check
makbigc Dec 15, 2018
29e820d
Modify _values_for_factorize in sparse.py
makbigc Dec 15, 2018
19621e6
Test merge with EA as key and open a special case
makbigc Dec 15, 2018
a69f7b7
Add factorize method in period.py
makbigc Dec 16, 2018
529f88d
Merge remote-tracking branch 'upstream/master' into makbigc-bug-fix-2…
TomAugspurger Dec 18, 2018
61f5e15
Updates
TomAugspurger Dec 18, 2018
6bae6ca
Reorganize the if / elifs
TomAugspurger Dec 18, 2018
File filter

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
Merge remote-tracking branch 'upstream/master' into makbigc-bug-fix-2… 
…3020
  • Loading branch information
@TomAugspurger
TomAugspurger committed Dec 18, 2018
commit 529f88dff4674bb9630d2ccfade01b5e7c6907b4
2 changes: 1 addition & 1 deletion doc/source/whatsnew/v0.24.0.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1051,8 +1051,8 @@ update the ``ExtensionDtype._metadata`` tuple to match the signature of your
- :meth:`DataFrame.stack` no longer converts to object dtype for DataFrames where each column has the same extension dtype. The output Series will have the same dtype as the columns (:issue:`23077`).
- :meth:`Series.unstack` and :meth:`DataFrame.unstack` no longer convert extension arrays to object-dtype ndarrays. Each column in the output ``DataFrame`` will now have the same dtype as the input (:issue:`23077`).
- Bug when grouping :meth:`Dataframe.groupby()` and aggregating on ``ExtensionArray`` it was not returning the actual ``ExtensionArray`` dtype (:issue:`23227`).
- A default repr for :class:`ExtensionArray` is now provided (:issue:`23601`).
- Bug in :func:`pandas.merge` when merging on an Integer extension array (:issue:`23020`)
- A default repr for :class:`pandas.api.extensions.ExtensionArray` is now provided (:issue:`23601`).

.. _whatsnew_0240.api.incompatibilities:

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.