Remove a few more uses of .format

Author: Armavica

pymc/distributions/shape_utils.py

@@ -141,11 +141,8 @@ def broadcast_dist_samples_shape(shapes, size=None):
     if size is None:
         broadcasted_shape = np.broadcast_shapes(*shapes)
         if broadcasted_shape is None:
-            raise ValueError(
-                "Cannot broadcast provided shapes {} given size: {}".format(
-                    ", ".join([f"{s}" for s in shapes]), size
-                )
-            )
+            tmp = ", ".join([f"{s}" for s in shapes])
+            raise ValueError(f"Cannot broadcast provided shapes {tmp} given size: {size}")
         return broadcasted_shape
     shapes = [_check_shape_type(s) for s in shapes]
     _size = to_tuple(size)
@@ -154,11 +151,8 @@ def broadcast_dist_samples_shape(shapes, size=None):
     try:
         broadcast_shape = np.broadcast_shapes(*sp_shapes)
     except ValueError:
-        raise ValueError(
-            "Cannot broadcast provided shapes {} given size: {}".format(
-                ", ".join([f"{s}" for s in shapes]), size
-            )
-        )
+        tmp = ", ".join([f"{s}" for s in shapes])
+        raise ValueError(f"Cannot broadcast provided shapes {tmp} given size: {size}")
     broadcastable_shapes = []
     for shape, sp_shape in zip(shapes, sp_shapes):
         if _size == shape[: len(_size)]:

pymc/gp/util.py

@@ -143,9 +143,8 @@ def getter(self):
                 value = getattr(self, name, None)
                 if value is None:
                     raise AttributeError(
-                        "'{}' not set.  Provide as argument "
-                        "to condition, or call 'prior' "
-                        "first".format(name.lstrip("_"))
+                        f"'{name.lstrip('_')}' not set.  Provide as argument "
+                        "to condition, or call 'prior' first"
                     )
                 else:
                     return value

pymc/printing.py

@@ -65,12 +65,11 @@ def str_for_dist(
             else r"\\operatorname{Unknown}"
         )
         if include_params:
+            params = ",~".join([d.strip("$") for d in dist_args])
             if print_name:
-                return r"${} \sim {}({})$".format(
-                    print_name, op_name, ",~".join([d.strip("$") for d in dist_args])
-                )
+                return rf"${print_name} \sim {op_name}({params})$"
             else:
-                return r"${}({})$".format(op_name, ",~".join([d.strip("$") for d in dist_args]))
+                return rf"${op_name}({params})$"
 
         else:
             if print_name:
@@ -83,10 +82,11 @@ def str_for_dist(
             dist.owner.op._print_name[0] if hasattr(dist.owner.op, "_print_name") else "Unknown"
         )
         if include_params:
+            params = ", ".join(dist_args)
             if print_name:
-                return r"{} ~ {}({})".format(print_name, dist_name, ", ".join(dist_args))
+                return rf"{print_name} ~ {dist_name}({params})"
             else:
-                return r"{}({})".format(dist_name, ", ".join(dist_args))
+                return rf"{dist_name}({params})"
         else:
             if print_name:
                 return rf"{print_name} ~ {dist_name}"