Enable binary operations between scalars and columns of differing decimal types

python/cudf/cudf/core/column/decimal.py

@@ -77,6 +77,7 @@ def _binaryop(self, other: ColumnBinaryOperand, op: str):
             return NotImplemented
         lhs, rhs = (other, self) if reflect else (self, other)
 
+
         # Binary Arithmetics between decimal columns. `Scale` and `precision`
         # are computed outside of libcudf
         if op in {"__add__", "__sub__", "__mul__", "__div__"}:
@@ -144,22 +145,20 @@ def normalize_binop_value(self, other):
             elif not isinstance(self.dtype, other.dtype.__class__):
                 # This branch occurs if we have a DecimalBaseColumn of a
                 # different size (e.g. 64 instead of 32).
-                if (
-                    self.dtype.precision == other.dtype.precision
-                    and self.dtype.scale == other.dtype.scale
-                ):
+                if _same_precision_and_scale(self.dtype, other.dtype):
                     other = other.astype(self.dtype)
-
             return other
         if isinstance(other, cudf.Scalar) and isinstance(
             # TODO: Should it be possible to cast scalars of other numerical
             # types to decimal?
             other.dtype,
             cudf.core.dtypes.DecimalDtype,
         ):
+            if _same_precision_and_scale(self.dtype, other.dtype):
+                other = other.astype(self.dtype)
             return other
         elif is_scalar(other) and isinstance(other, (int, Decimal)):
-            return cudf.Scalar(Decimal(other))
+            return cudf.Scalar(Decimal(other), dtype=self.dtype)
         return NotImplemented
 
     def _decimal_quantile(
@@ -404,3 +403,7 @@ def _get_decimal_type(lhs_dtype, rhs_dtype, op):
     )
     precision = min(cudf.Decimal128Dtype.MAX_PRECISION, max_precision)
     return cudf.Decimal128Dtype(precision=precision, scale=scale)
+
+
+def _same_precision_and_scale(lhs: DecimalDtype, rhs: DecimalDtype) -> bool:
+    return lhs.precision == rhs.precision and lhs.scale == rhs.scale

python/cudf/cudf/tests/test_binops.py

@@ -3012,10 +3012,10 @@ def decimal_series(input, dtype):
     ],
 )
 @pytest.mark.parametrize("reflected", [True, False])
-@pytest_xfail(
-    reason="binop operations not supported for different bit-width "
-    "decimal types"
-)
+# @pytest_xfail(
+#     reason="binop operations not supported for different bit-width "
+#     "decimal types"
+# )
 def test_binops_decimal_scalar_compare(args, reflected):
     """
     Tested compare operations: