Fix for issue pandas-dev#28283: Ensure DataFrame.eval calls __finalize__

Author: root

pandas/core/frame.py

@@ -4628,27 +4628,36 @@ def query(self, expr: str, *, inplace: bool = False, **kwargs) -> DataFrame | No
             return result
 
     @overload
-    def eval(self, expr: str, *, inplace: Literal[False] = ..., **kwargs) -> Any: ...
 
-    @overload
-    def eval(self, expr: str, *, inplace: Literal[True], **kwargs) -> None: ...
+def eval(self, expr: str, *, inplace: bool = False, **kwargs) -> Any | None:
+    """
+    Evaluate a string describing operations on DataFrame columns.
 
-    def eval(self, expr: str, *, inplace: bool = False, **kwargs) -> Any | None:
-        """
-        Evaluate a string describing operations on DataFrame columns.
+    Operates on columns only, not specific rows or elements.  This allows
+    `eval` to run arbitrary code, which can make you vulnerable to code
+    injection if you pass user input to this function.
 
-        Operates on columns only, not specific rows or elements.  This allows
-        `eval` to run arbitrary code, which can make you vulnerable to code
-        injection if you pass user input to this function.
+    Parameters
+    ----------
+    expr : str
+        The expression string to evaluate.
+    inplace : bool, default False
+        If the expression contains an assignment, whether to perform the
+        operation inplace and mutate the existing DataFrame. Otherwise,
+        a new DataFrame is returned.
+    """
+    from pandas.core.computation.eval import eval as _eval
+
+    if not isinstance(expr, str):
+        msg = f"expr must be a string to be evaluated, {type(expr)} given"
+        raise ValueError(msg)
+
+    # Ensure __finalize__ is called to propagate metadata
+    result = super().eval(expr, inplace=inplace, **kwargs)
+    if not inplace:
+        result = result.__finalize__(self, method="eval")
+    return result
 
-        Parameters
-        ----------
-        expr : str
-            The expression string to evaluate.
-        inplace : bool, default False
-            If the expression contains an assignment, whether to perform the
-            operation inplace and mutate the existing DataFrame. Otherwise,
-            a new DataFrame is returned.
         **kwargs
             See the documentation for :func:`eval` for complete details
             on the keyword arguments accepted by