Skip to content

BUG: fix non-existent variable in NDFrame.interpolate #29142

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 11 commits into from
Oct 25, 2019
Merged

BUG: fix non-existent variable in NDFrame.interpolate #29142

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
0259cf5
BUG: Fix NDFrame.interpolate non-existent variable #29132
ellequelle Oct 21, 2019
66a1b1a
Merge remote-tracking branch 'upstream/master' into interpbug
ellequelle Oct 22, 2019
b63920b
Removed unreachable else statement in NDFrame.interpolate
ellequelle Oct 22, 2019
e1191b6
Added a test for issue #29142
ellequelle Oct 22, 2019
f8feeb9
Edited test for issue #29142
ellequelle Oct 22, 2019
7331d2d
Adding issue number to test
ellequelle Oct 22, 2019
0a4067b
Edit whatsnew file to reference a user facing method instead of NDFrame
ellequelle Oct 22, 2019
8231559
Added assert_frame_equal to check object returned by interpolate
ellequelle Oct 23, 2019
499147a
Moving GH #29142 to same file as other DataFrame.interpolate tests
ellequelle Oct 23, 2019
d2bb8a0
Moved interpolate axis name tests to a new parametrized function
ellequelle Oct 25, 2019
aae361d
Added RuntimeWarning assertion to test_fillna_categorical_nan
ellequelle Oct 25, 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
1 change: 1 addition & 0 deletions doc/source/whatsnew/v1.0.0.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,6 +249,7 @@ Bug fixes
~~~~~~~~~

- Bug in :meth:`DataFrame.to_html` when using ``formatters=<list>`` and ``max_cols`` together. (:issue:`25955`)
- Bug in :func:`pandas.core.generic.NDFrame.interpolate` where specifying axis by name references variable before it is assigned (:issue:`29132`)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are user facing, so we shouldn't reference NDFrame. Instead, refer to just :meth:`DataFrame.interpolate`, even if it affects Series as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see!


Categorical
^^^^^^^^^^^
Expand Down
5 changes: 3 additions & 2 deletions pandas/core/generic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7023,14 +7023,15 @@ def interpolate(
"""
inplace = validate_bool_kwarg(inplace, "inplace")

axis = self._get_axis_number(axis)

if axis == 0:
ax = self._info_axis_name
_maybe_transposed_self = self
elif axis == 1:
_maybe_transposed_self = self.T
ax = 1
else:
_maybe_transposed_self = self

ax = _maybe_transposed_self._get_axis_number(ax)

if _maybe_transposed_self.ndim == 2:
Expand Down
5 changes: 5 additions & 0 deletions pandas/tests/generic/test_generic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -948,3 +948,8 @@ def test_deprecated_get_dtype_counts(self):
df = DataFrame([1])
with tm.assert_produces_warning(FutureWarning):
df.get_dtype_counts()

@pytest.mark.parametrize("axis", [0, 1, "index", "columns", "rows"])
def test_interpolate_axis_argument(self, axis):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reference issue number above this line as a comment.

# GH 29142
assert DataFrame([0]).interpolate(axis=axis)