BUG: DataFrame.(any|all) inconsistency

pandas/core/internals/managers.py

@@ -18,6 +18,7 @@
 )
 from pandas.core.dtypes.common import (
     DT64NS_DTYPE,
+    is_bool_dtype,
     is_datetimelike_v_numeric,
     is_extension_array_dtype,
     is_list_like,
@@ -688,8 +689,9 @@ def get_bool_data(self, copy: bool = False) -> "BlockManager":
         copy : bool, default False
             Whether to copy the blocks
         """
-        self._consolidate_inplace()
-        return self._combine([b for b in self.blocks if b.is_bool], copy)
+        # Note: use is_bool_dtype instead of blk.is_bool to exclude
+        #  object-dtype blocks containing all-bool entries.
+        return self._combine([b for b in self.blocks if is_bool_dtype(b.dtype)], copy)
 
     def get_numeric_data(self, copy: bool = False) -> "BlockManager":
         """
@@ -698,7 +700,6 @@ def get_numeric_data(self, copy: bool = False) -> "BlockManager":
         copy : bool, default False
             Whether to copy the blocks
         """
-        self._consolidate_inplace()
         return self._combine([b for b in self.blocks if b.is_numeric], copy)
 
     def _combine(self, blocks: List[Block], copy: bool = True) -> "BlockManager":
@@ -875,12 +876,7 @@ def to_dict(self, copy: bool = True):
         -------
         values : a dict of dtype -> BlockManager
 
-        Notes
-        -----
-        This consolidates based on str(dtype)
         """
-        self._consolidate_inplace()
-
         bd: Dict[str, List[Block]] = {}
         for b in self.blocks:
             bd.setdefault(str(b.dtype), []).append(b)

pandas/tests/internals/test_internals.py

@@ -700,6 +700,20 @@ def test_get_bool_data(self):
             np.array([True, False, True]),
         )
 
+    def test_get_bool_data_consistency(self):
+        # GH#34918
+        ser = pd.Series([True, False, True], dtype=object)
+        ser2 = pd.Series(["A", "B", "C"])
+        df = ser.to_frame("A")
+
+        expected = pd.DataFrame(index=ser.index)
+        bd1 = df._get_bool_data()
+        tm.assert_frame_equal(bd1, expected)
+
+        df["B"] = ser2
+        bd2 = df._get_bool_data()
+        tm.assert_frame_equal(bd2, expected)
+
     def test_unicode_repr_doesnt_raise(self):
         repr(create_mgr("b,\u05d0: object"))