BUG: Validate numeric_only parameter in groupby aggregations

Author: Zrahay

pandas/core/groupby/groupby.py

@@ -1754,44 +1754,48 @@ def _cython_agg_general(
         **kwargs,
     ):
         # Note: we never get here with how="ohlc" for DataFrameGroupBy;
-        #  that goes through SeriesGroupBy
+        # that goes through SeriesGroupBy
 
-        data = self._get_data_to_aggregate(numeric_only=numeric_only, name=how)
+        # Check to confirm numeric_only is fed either True or False and no other data type
+        if(isinstance(numeric_only, bool)):
+            data = self._get_data_to_aggregate(numeric_only=numeric_only, name=how)
 
-        def array_func(values: ArrayLike) -> ArrayLike:
-            try:
-                result = self._grouper._cython_operation(
-                    "aggregate",
-                    values,
-                    how,
-                    axis=data.ndim - 1,
-                    min_count=min_count,
-                    **kwargs,
-                )
-            except NotImplementedError:
-                # generally if we have numeric_only=False
-                # and non-applicable functions
-                # try to python agg
-                # TODO: shouldn't min_count matter?
-                # TODO: avoid special casing SparseArray here
-                if how in ["any", "all"] and isinstance(values, SparseArray):
-                    pass
-                elif alt is None or how in ["any", "all", "std", "sem"]:
-                    raise  # TODO: re-raise as TypeError?  should not be reached
-            else:
-                return result
+            def array_func(values: ArrayLike) -> ArrayLike:
+                try:
+                    result = self._grouper._cython_operation(
+                        "aggregate",
+                        values,
+                        how,
+                        axis=data.ndim - 1,
+                        min_count=min_count,
+                        **kwargs,
+                    )
+                except NotImplementedError:
+                    # generally if we have numeric_only=False
+                    # and non-applicable functions
+                    # try to python agg
+                    # TODO: shouldn't min_count matter?
+                    # TODO: avoid special casing SparseArray here
+                    if how in ["any", "all"] and isinstance(values, SparseArray):
+                        pass
+                    elif alt is None or how in ["any", "all", "std", "sem"]:
+                        raise  # TODO: re-raise as TypeError?  should not be reached
+                else:
+                    return result
 
-            assert alt is not None
-            result = self._agg_py_fallback(how, values, ndim=data.ndim, alt=alt)
-            return result
+                assert alt is not None
+                result = self._agg_py_fallback(how, values, ndim=data.ndim, alt=alt)
+                return result
 
-        new_mgr = data.grouped_reduce(array_func)
-        res = self._wrap_agged_manager(new_mgr)
-        if how in ["idxmin", "idxmax"]:
-            # mypy expects how to be Literal["idxmin", "idxmax"].
-            res = self._wrap_idxmax_idxmin(res, how=how, skipna=kwargs["skipna"])  # type: ignore[arg-type]
-        out = self._wrap_aggregated_output(res)
-        return out
+            new_mgr = data.grouped_reduce(array_func)
+            res = self._wrap_agged_manager(new_mgr)
+            if how in ["idxmin", "idxmax"]:
+                # mypy expects how to be Literal["idxmin", "idxmax"].
+                res = self._wrap_idxmax_idxmin(res, how=how, skipna=kwargs["skipna"])  # type: ignore[arg-type]
+            out = self._wrap_aggregated_output(res)
+            return out
+        else:
+            raise ValueError("numeric_only accepts only Boolean values")
 
     def _cython_transform(self, how: str, numeric_only: bool = False, **kwargs):
         raise AbstractMethodError(self)