-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
CLN GH23123 Move SparseArray to arrays #23147
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
Changes from 9 commits
233a284
96cd3d5
c9bfeae
f63ac12
36285c1
ab60797
e8808e0
54ed5bc
b261f85
1d0b50b
fe53b50
120bc5e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,7 +19,8 @@ | |
""" | ||
|
||
from numpy import ndarray | ||
from pandas.util._validators import (validate_args, validate_kwargs, | ||
from pandas.util._validators import (validate_args, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is there a reason you are changing unrelated things? (even formatting) |
||
validate_kwargs, | ||
validate_args_and_kwargs) | ||
from pandas.errors import UnsupportedFunctionCall | ||
from pandas.core.dtypes.common import is_integer, is_bool | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -523,7 +523,7 @@ def _add_delta(self, delta): | |
The result's name is set outside of _add_delta by the calling | ||
method (__add__ or __sub__) | ||
""" | ||
from pandas.core.arrays.timedeltas import TimedeltaArrayMixin | ||
from pandas.core.arrays import TimedeltaArrayMixin | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Were these changes here before mine? I may revert them. |
||
|
||
if isinstance(delta, (Tick, timedelta, np.timedelta64)): | ||
new_values = self._add_delta_td(delta) | ||
|
@@ -818,7 +818,7 @@ def to_period(self, freq=None): | |
pandas.PeriodIndex: Immutable ndarray holding ordinal values | ||
pandas.DatetimeIndex.to_pydatetime: Return DatetimeIndex as object | ||
""" | ||
from pandas.core.arrays.period import PeriodArrayMixin | ||
from pandas.core.arrays import PeriodArrayMixin | ||
|
||
if self.tz is not None: | ||
warnings.warn("Converting to PeriodArray/Index representation " | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
# pylint: disable=W0611 | ||
# flake8: noqa | ||
from pandas.core.arrays.sparse.array import SparseArray | ||
from pandas.core.arrays.sparse.series import SparseSeries | ||
from pandas.core.arrays.sparse.frame import SparseDataFrame | ||
from pandas.core.arrays.sparse.dtype import SparseDtype |
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, should have reverted this too :/ Oh well.