10000 CLN: handle EAs and fast path (no bounds checking) in safe_sort by jorisvandenbossche · Pull Request #25696 · pandas-dev/pandas · GitHub
[go: up one dir, main page]

Skip to content

CLN: handle EAs and fast path (no bounds checking) in safe_sort #25696

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 17 commits into from
May 7, 2019
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
7356997
BUG: fix usage of na_sentinel with sort=True in factorize()
jorisvandenbossche Mar 7, 2019
e1ab3a4
fix dtype
jorisvandenbossche Mar 11, 2019
a9c880e
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Mar 11, 2019
db30797
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Mar 12, 2019
ba944eb
Attempt to include it in safe_sort
jorisvandenbossche Mar 12, 2019
c6203cb
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Mar 12, 2019
d70b447
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Apr 5, 2019
fdf330a
feedback Jeff
jorisvandenbossche Apr 5, 2019
b08ea6d
add tests for safe_sort
jorisvandenbossche Apr 5, 2019
9de26fc
additional test for other na_sentinel in case of out of bound indices
jorisvandenbossche Apr 5, 2019
bcb8c7e
additional test for EA with custom na_sentinel
jorisvandenbossche Apr 5, 2019
13f6706
update factorize test for EAs with custom na_sentinel (which now work…
jorisvandenbossche Apr 5, 2019
8db84e7
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Apr 5, 2019
d0cef9e
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche Apr 11, 2019
5157e89
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
jorisvandenbossche May 6, 2019
e350641
linting
jorisvandenbossche May 6, 2019
151aa6a
more linting
jorisvandenbossche May 6, 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
Merge remote-tracking branch 'upstream/master' into factorize-na-sent…
…inel2
  • Loading branch information
jorisvandenbossche committed May 6, 2019
commit 5157e89e4a79bd9e942dd6248657f36c64a167b0
4 changes: 1 addition & 3 deletions doc/source/whatsnew/v0.25.0.rst
Original file line number Diff line number Diff line change
Expand Up @@ -427,10 +427,8 @@ Sparse
Other
^^^^^

- Improved :class:`Timestamp` type checking in various datetime functions to prevent exceptions when using a subclassed `datetime` (:issue:`25851`)
- Bug in :class:`Series` and :class:`DataFrame` repr where ``np.datetime64('NaT')`` and ``np.timedelta64('NaT')`` with ``dtype=object`` would be represented as ``NaN`` (:issue:`25445`)
- Removed unused C functions from vendored UltraJSON implementation (:issue:`26198`)
- Bug in :func:`factorize` when passing an ``ExtensionArray`` with a custom ``na_sentinel`` (:issue:`25696`).
-


.. _whatsnew_0.250.contributors:
Expand Down
2 changes: 1 addition & 1 deletion pandas/tests/test_sorting.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
import pytest

from pandas import (
DataFrame, MultiIndex, Series, array, compat, concat, merge, to_datetime)
DataFrame, MultiIndex, Series, array, concat, merge, to_datetime)
from pandas.core import common as com
from pandas.core.sorting import (
decons_group_index, get_group_index, is_int64_overflow_possible,
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.
0