Small nits

keras-team · Nov 24, 2023 · 9620d23 · 9620d23
1 parent b4c3a0e
commit 9620d23
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 18 deletions.
diff --git a/keras/backend/jax/numpy.py b/keras/backend/jax/numpy.py
@@ -588,9 +588,9 @@ def pad(x, pad_width, mode="constant", constant_values=None):
     if constant_values is not None:
         if mode != "constant":
             raise ValueError(
-                "Argument 'constant_values' can only be "
-                "provided when mode == 'constant'. "
-                f"Received mode '{mode}'"
+                "Argument `constant_values` can only be "
+                "provided when `mode == 'constant'`. "
+                f"Received: mode={mode}"
             )
         kwargs["constant_values"] = constant_values
     return jnp.pad(x, pad_width, mode=mode, **kwargs)

diff --git a/keras/backend/numpy/numpy.py b/keras/backend/numpy/numpy.py
@@ -709,9 +709,9 @@ def pad(x, pad_width, mode="constant", constant_values=None):
     if constant_values is not None:
         if mode != "constant":
             raise ValueError(
-                "Argument 'constant_values' can only be "
-                "provided when mode == 'constant'. "
-                f"Received mode '{mode}'"
+                "Argument `constant_values` can only be "
+                "provided when `mode == 'constant'`. "
+                f"Received: mode={mode}"
             )
         kwargs["constant_values"] = constant_values
     return np.pad(x, pad_width, mode=mode, **kwargs)

diff --git a/keras/backend/tensorflow/numpy.py b/keras/backend/tensorflow/numpy.py
@@ -1065,9 +1065,9 @@ def pad(x, pad_width, mode="constant", constant_values=None):
     if constant_values is not None:
         if mode != "constant":
             raise ValueError(
-                "Argument 'constant_values' can only be "
-                "provided when mode == 'constant'. "
-                f"Received mode '{mode}'"
+                "Argument `constant_values` can only be "
+                "provided when `mode == 'constant'`. "
+                f"Received: mode={mode}"
             )
         kwargs["constant_values"] = constant_values
     return tfnp.pad(x, pad_width, mode=mode, **kwargs)

diff --git a/keras/backend/torch/numpy.py b/keras/backend/torch/numpy.py
@@ -954,9 +954,9 @@ def pad(x, pad_width, mode="constant", constant_values=None):
     if constant_values is not None:
         if mode != "constant":
             raise ValueError(
-                "Argument 'constant_values' can only be "
-                "provided when mode == 'constant'. "
-                f"Received mode '{mode}'"
+                "Argument `constant_values` can only be "
+                "provided when `mode == 'constant'`. "
+                f"Received: mode={mode}"
             )
         kwargs["value"] = constant_values
     x = convert_to_tensor(x)

diff --git a/keras/ops/numpy.py b/keras/ops/numpy.py
@@ -4190,8 +4190,8 @@ def _process_pad_width(self, pad_width):
         for i, pw in enumerate(pad_width):
             if len(pw) != first_len:
                 raise ValueError(
-                    "`pad_width` should be a list of tuples of length 2 or "
-                    f"1, but received {pad_width}."
+                    "`pad_width` should be a list of tuples of length "
+                    f"1 or 2. Received: pad_width={pad_width}"
                 )
             if len(pw) == 1:
                 pad_width[i] = (pw[0], pw[0])
@@ -4213,8 +4213,10 @@ def compute_output_spec(self, x, constant_values=None):
             pad_width = self.pad_width
         else:
             raise ValueError(
-                "`pad_width` must have the same length as `x.shape`, but "
-                f"received {len(self.pad_width)} and {len(x.shape)}."
+                "`pad_width` must have the same length as `x.shape`. "
+                f"Received: pad_width={self.pad_width} "
+                f"(of length {len(self.pad_width)}) and x.shape={x.shape} "
+                f"(of length {len(x.shape)})"
             )
 
         for i in range(len(output_shape)):

diff --git a/keras/ops/numpy_test.py b/keras/ops/numpy_test.py
@@ -3553,8 +3553,8 @@ def test_pad(self, dtype, mode, constant_values):
             if constant_values is not None:
                 with self.assertRaisesRegex(
                     ValueError,
-                    "Argument 'constant_values' can only be "
-                    "provided when mode == 'constant'",
+                    "Argument `constant_values` can only be "
+                    "provided when `mode == 'constant'`",
                 ):
                     knp.pad(
                         x, pad_width, mode=mode, constant_values=constant_values