Skip to content

Commit 5c955cb

Browse files
gfyoungjorisvandenbossche
authored andcommitted
CLN: Drop lags parameter from Panel.shift (#14041)
1 parent a01e58f commit 5c955cb

File tree

3 files changed

+2
-16
lines changed

3 files changed

+2
-16
lines changed

doc/source/whatsnew/v0.19.0.txt

+1
Original file line numberDiff line numberDiff line change
@@ -891,6 +891,7 @@ Removal of prior version deprecations/changes
891891
- ``pd.Categorical`` has dropped setting of the ``ordered`` attribute directly in favor of the ``set_ordered`` method (:issue:`13671`)
892892
- ``pd.Categorical`` has dropped the ``levels`` attribute in favour of ``categories`` (:issue:`8376`)
893893
- ``DataFrame.to_sql()`` has dropped the ``mysql`` option for the ``flavor`` parameter (:issue:`13611`)
894+
- ``Panel.shift()`` has dropped the ``lags`` parameter in favour of ``periods`` (:issue:`14041`)
894895
- ``pd.Index`` has dropped the ``diff`` method in favour of ``difference`` (:issue:`13669`)
895896

896897
- ``Series.to_csv`` has dropped the ``nanRep`` parameter in favor of ``na_rep`` (:issue:`13804`)

pandas/core/panel.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
from pandas.core.ops import _op_descriptions
3636
from pandas.core.series import Series
3737
from pandas.tools.util import cartesian_product
38-
from pandas.util.decorators import (deprecate, Appender, deprecate_kwarg)
38+
from pandas.util.decorators import (deprecate, Appender)
3939

4040
_shared_doc_kwargs = dict(
4141
axes='items, major_axis, minor_axis',
@@ -1234,7 +1234,6 @@ def count(self, axis='major'):
12341234

12351235
return self._wrap_result(result, axis)
12361236

1237-
@deprecate_kwarg(old_arg_name='lags', new_arg_name='periods')
12381237
def shift(self, periods=1, freq=None, axis='major'):
12391238
"""
12401239
Shift index by desired number of periods with an optional time freq.

pandas/tests/test_panel.py

-14
Original file line numberDiff line numberDiff line change
@@ -881,20 +881,6 @@ def setUp(self):
881881
self.panel.minor_axis.name = None
882882
self.panel.items.name = None
883883

884-
def test_panel_warnings(self):
885-
with tm.assert_produces_warning(FutureWarning):
886-
shifted1 = self.panel.shift(lags=1)
887-
888-
with tm.assert_produces_warning(False):
889-
shifted2 = self.panel.shift(periods=1)
890-
891-
tm.assert_panel_equal(shifted1, shifted2)
892-
893-
with tm.assert_produces_warning(False):
894-
shifted3 = self.panel.shift()
895-
896-
tm.assert_panel_equal(shifted1, shifted3)
897-
898884
def test_constructor(self):
899885
# with BlockManager
900886
wp = Panel(self.panel._data)

0 commit comments

Comments
 (0)