Skip to content

Commit

Permalink
BUG: rolling.cov with multi-index columns should presever the MI
Browse files Browse the repository at this point in the history
xref #16814
  • Loading branch information
jreback committed Jul 4, 2017
1 parent 9e55af2 commit 42b9827
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 13 deletions.
6 changes: 3 additions & 3 deletions pandas/core/window.py
Original file line number Diff line number Diff line change
Expand Up @@ -836,7 +836,7 @@ def count(self):

return self._wrap_results(results, blocks, obj)

_shared_docs['apply'] = dedent("""
_shared_docs['apply'] = dedent(r"""
%(name)s function apply
Parameters
Expand Down Expand Up @@ -1945,8 +1945,8 @@ def dataframe_from_int_dict(data, frame_template):
# reset our index names to arg1 names
# reset our column names to arg2 names
# careful not to mutate the original names
result.columns = Index(result.columns).set_names(
arg2.columns.name)
result.columns = result.columns.set_names(
arg2.columns.names)
result.index = result.index.set_names(
arg1.index.names + arg1.columns.names)

Expand Down
9 changes: 0 additions & 9 deletions pandas/tests/indexes/test_multi.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,15 +61,6 @@ def f():

tm.assert_raises_regex(ValueError, 'The truth value of a', f)

def test_multi_index_names(self):

# GH 16789
cols = pd.MultiIndex.from_product([['A', 'B'], ['C', 'D', 'E']],
names=['1', '2'])
df = pd.DataFrame(np.ones((10, 6)), columns=cols)
rolling_result = df.rolling(3).cov()
assert rolling_result.index.names == [None, '1', '2']

def test_labels_dtypes(self):

# GH 8456
Expand Down
14 changes: 13 additions & 1 deletion pandas/tests/test_window.py
Original file line number Diff line number Diff line change
Expand Up @@ -455,6 +455,17 @@ def tests_empty_df_rolling(self, roller):
result = DataFrame(index=pd.DatetimeIndex([])).rolling(roller).sum()
tm.assert_frame_equal(result, expected)

def test_multi_index_names(self):

# GH 16789
cols = pd.MultiIndex.from_product([['A', 'B'], ['C', 'D', 'E']],
names=['1', '2'])
df = pd.DataFrame(np.ones((10, 6)), columns=cols)
result = df.rolling(3).cov()

tm.assert_index_equal(result.columns, df.columns)
assert result.index.names == [None, '1', '2']


class TestExpanding(Base):

Expand Down Expand Up @@ -501,9 +512,10 @@ def test_numpy_compat(self):
'expander',
[1, pytest.mark.xfail(
reason='GH 16425 expanding with offset not supported')('1s')])
def tests_empty_df_expanding(self, expander):
def test_empty_df_expanding(self, expander):
# GH 15819 Verifies that datetime and integer expanding windows can be
# applied to empty DataFrames

expected = DataFrame()
result = DataFrame().expanding(expander).sum()
tm.assert_frame_equal(result, expected)
Expand Down

0 comments on commit 42b9827

Please sign in to comment.