8000 BUG: Fix exceptions when Series.interpolate's `order` parameter is missing or invalid by nmusolino · Pull Request #25246 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

BUG: Fix exceptions when Series.interpolate's order parameter is missing or invalid #25246

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
Feb 11, 2019
Merged

BUG: Fix exceptions when Series.interpolate's order parameter is missing or invalid #25246

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
de5b5d2
BUG: raise accurate exception from Series.interpolate (#24014)
nmusolino Feb 9, 2019
5bd3d4d
Actually validate `order` before use in spline
nmusolino Feb 9, 2019
c4ef8e9
Remove unnecessary check and dead code
nmusolino Feb 9, 2019
f6b8907
Clean up comparison/tests based on feedback
nmusolino Feb 10, 2019
c4f3b6d
Include invalid order value in exception
nmusolino Feb 10, 2019
2d5ec0d
Check for NaN order in spline validation
nmusolino Feb 10, 2019
097b9e4
Add whatsnew entry for bug fix
nmusolino Feb 11, 2019
becc36d
CLN: Make unit tests assert one error at a time
nmusolino Feb 11, 2019
33a0de2
CLN: break test into distinct test case
nmusolino Feb 11, 2019
24cd9f5
PEP8 fix in test module
nmusolino Feb 11, 2019
69dd362
CLN: Test fixture for interpolate methods
nmusolino Feb 11, 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
CLN: break test into distinct test case
  • Loading branch information
@nmusolino
nmusolino committed Feb 11, 2019
commit 33a0de29ef8bad1ec8f66935de8916ef23bfed9b
11 changes: 5 additions & 6 deletions pandas/tests/series/test_missing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -882,12 +882,11 @@ def test_interpolate(self, datetime_series, string_series):
time_interp = ord_ts_copy.interpolate(method='time')
tm.assert_series_equal(time_interp, ord_ts)

# try time interpolation on a non-TimeSeries
# Only raises ValueError if there are NaNs.
non_ts = string_series.copy()
non_ts[0] = np.NaN
msg = ("time-weighted interpolation only works on Series or DataFrames"
" with a DatetimeIndex")
def test_interpolate_time_raises_for_non_timeseries(self):
# When method='time' is used on a non-TimeSeries that contains a null
# value, a ValueError should be raised.
non_ts = Series([0, 1, 2, np.NaN])
msg = "time-weighted interpolation only works on Series.* with a DatetimeIndex"
with pytest.raises(ValueError, match=msg):
non_ts.interpolate(method='time')

Expand Down
0