BUG: added series type to wrap_result for empty DataFrame

Author: discort

doc/source/whatsnew/v0.19.0.txt

@@ -1563,3 +1563,4 @@ Bug Fixes
 - ``PeriodIndex`` can now accept ``list`` and ``array`` which contains ``pd.NaT`` (:issue:`13430`)
 - Bug in ``df.groupby`` where ``.median()`` returns arbitrary values if grouped dataframe contains empty bins (:issue:`13629`)
 - Bug in ``Index.copy()`` where ``name`` parameter was ignored (:issue:`14302`)
+- Bug in ``_downsample()``. Inconsistent return type on resample of empty DataFrame (:issue:`14962`)

pandas/core/resample.py

@@ -697,12 +697,8 @@ def _downsample(self, how, **kwargs):
         if not len(ax):
             # reset to the new freq
             obj = obj.copy()
-            if how == "size" and isinstance(obj, pd.DataFrame):
-                obj = obj.groupby(
-                    self.grouper, axis=self.axis).aggregate(how, **kwargs)
-
             obj.index.freq = self.freq
-            return obj
+            return self._wrap_result(obj)
 
         # do we have a regular frequency
         if ax.freq is not None or ax.inferred_freq is not None:
@@ -773,6 +769,15 @@ def _wrap_result(self, result):
         # convert if needed
         if self.kind == 'period' and not isinstance(result.index, PeriodIndex):
             result.index = result.index.to_period(self.freq)
+
+        # Make consistent type of result. GH14962
+        if not len(self.ax):
+            grouper = BinGrouper([], result.index)
+            grouped = self._selected_obj.groupby(grouper)
+            result = pd.Series([],
+                               index=result.index,
+                               name=grouped.name,
+                               dtype='int64')
         return result
 
 

pandas/tests/test_resample.py

@@ -773,8 +773,18 @@ def test_resample_empty_series(self):
                 expected = s.copy()
                 expected.index = s.index._shallow_copy(freq=freq)
                 assert_index_equal(result.index, expected.index)
-                assert result.index.freq == expected.index.freq
-                assert_series_equal(result, expected, check_dtype=False)
+
+                self.assertEqual(result.index.freq, expected.index.freq)
+
+                if (method == 'size' and
+                    isinstance(result.index, PeriodIndex) and
+                        freq in ['M', 'D']):
+                    # GH12871 - TODO: name should propagate, but currently
+                    # doesn't on lower / same frequency with PeriodIndex
+                    assert_series_equal(result, expected, check_dtype=False)
+
+                else:
+                    assert_series_equal(result, expected, check_dtype=False)
 
     def test_resample_empty_dataframe(self):
         # GH13212
@@ -783,11 +793,11 @@ def test_resample_empty_dataframe(self):
 
         for freq in ['M', 'D', 'H']:
             # count retains dimensions too
-            methods = downsample_methods + ['count']
+            methods = downsample_methods + upsample_methods
             for method in methods:
                 result = getattr(f.resample(freq), method)()
 
-                expected = f.copy()
+                expected = pd.Series([])
                 expected.index = f.index._shallow_copy(freq=freq)
                 assert_index_equal(result.index, expected.index)
                 assert result.index.freq == expected.index.freq
@@ -850,11 +860,13 @@ def test_resample_loffset_arg_type(self):
 
     def test_resample_empty_dataframe_with_size(self):
         # GH 14962
-        df1 = pd.DataFrame(dict(a=range(100)),
-                           index=pd.date_range('1/1/2000', periods=100, freq="M"))
-        df2 = df1[df1.a < 0]
-        result = df2.resample("Q").size()
-        assertIsInstance(result, pd.Series)
+        index = pd.DatetimeIndex([], freq='M')
+        df = pd.DataFrame([], index=index)
+
+        for freq in ['M', 'D', 'H']:
+            result = df.resample(freq).size()
+            expected = pd.Series([], index=index, dtype='int64')
+            assert_series_equal(result, expected)
 
 
 class TestDatetimeIndex(Base):