Update f-string (RUF073)

.basedpyright/baseline.json

@@ -276,8 +276,8 @@
             {
                 "code": "reportUnusedImport",
                 "range": {
-                    "startColumn": 48,
-                    "endColumn": 88,
+                    "startColumn": 12,
+                    "endColumn": 52,
                     "lineCount": 1
                 }
             },
@@ -6687,14 +6687,6 @@
             }
         ],
         "./grudge/reductions.py": [
-            {
-                "code": "reportUnknownVariableType",
-                "range": {
-                    "startColumn": 4,
-                    "endColumn": 22,
-                    "lineCount": 1
-                }
-            },
             {
                 "code": "reportUnknownParameterType",
                 "range": {

examples/wave/wave-op-var-velocity.py

@@ -27,11 +27,11 @@
 import logging
 
 import numpy as np
-import numpy.linalg as la  # noqa
+import numpy.linalg as la  # noqa: F401
 
 import pyopencl as cl
 import pyopencl.tools as cl_tools
-from meshmode.mesh import BTAG_ALL, BTAG_NONE  # noqa
+from meshmode.mesh import BTAG_ALL, BTAG_NONE  # noqa: F401
 from pytools import obj_array
 
 import grudge.geometry as geo

grudge/array_context.py

@@ -62,7 +62,9 @@
     try:
         # Crude check if we have the correct loopy branch
         # (https://github.com/kaushikcfd/loopy/tree/pytato-array-context-transforms)
-        from loopy.transform.loop_fusion import get_kennedy_unweighted_fusion_candidates  # noqa
+        from loopy.transform.loop_fusion import (
+            get_kennedy_unweighted_fusion_candidates,  # noqa: F401
+        )
     except ImportError:
         warn("Your loopy and meshmode branches are mismatched. "
              "Please make sure that you have the "
@@ -645,7 +647,7 @@ def get_reasonable_array_context_class(
             if not _HAVE_FUSION_ACTX:
                 warn("No device-parallel actx available, execution will be slow. "
                      "Please make sure you have the right branches for loopy "
-                     "(https://github.com/kaushikcfd/loopy/tree/pytato-array-context-transforms) "  # noqa
+                     "(https://github.com/kaushikcfd/loopy/tree/pytato-array-context-transforms) "  # noqa: E501
                      "and meshmode "
                      "(https://github.com/kaushikcfd/meshmode/tree/pytato-array-context-transforms).",
                      stacklevel=1)

grudge/geometry/metrics.py

@@ -410,7 +410,7 @@ def inverse_first_fundamental_form(
                  obj_array.new_1d([-F, E])]
             )
         else:
-            raise ValueError(f"{dim}D surfaces not supported" % dim)
+            raise ValueError(f"{dim}D surfaces not supported")
 
     return inv_form1
 

grudge/trace_pair.py

@@ -426,7 +426,7 @@ def _sym_tag_to_num_tag(comm_tag: Hashable | None, base_tag: int) -> int:
 
     # FIXME: This isn't guaranteed to be correct.
     # See here for discussion:
-    # - https://github.com/illinois-ceesd/mirgecom/issues/617#issuecomment-1057082716  # noqa
+    # - https://github.com/illinois-ceesd/mirgecom/issues/617#issuecomment-1057082716
     # - https://github.com/inducer/grudge/pull/222
     # Since only 1 communication can be pending for a given tag at a time,
     # this does not matter currently. See https://github.com/inducer/grudge/issues/223

test/test_grudge.py

@@ -873,7 +873,7 @@ def rhs(t, u, adv_operator=adv_operator):
 
             if visualize:
                 vis.write_vtk_file(
-                    f"fld-{mesh_par}-{step:04d}vtu" % (mesh_par, step),
+                    f"fld-{mesh_par}-{step:04d}vtu",
                     [("u", u)]
                 )