Upgrade to fstrings

Author: VeckoTheGecko

docs/examples/example_dask_chunk_OCMs.py

@@ -247,7 +247,7 @@ def test_pop(mode, chunk_mode):
     filenames = str(data_folder / "t.x1_SAMOC_flux.1690*.nc")
     variables = {"U": "UVEL", "V": "VVEL", "W": "WVEL"}
     timestamps = np.expand_dims(
-        np.array([np.datetime64("2000-%.2d-01" % m) for m in range(1, 7)]), axis=1
+        np.array([np.datetime64(f"2000-{m:02d}-01") for m in range(1, 7)]), axis=1
     )
     dimensions = {"lon": "ULON", "lat": "ULAT", "depth": "w_dep"}
     chs = False

docs/examples/example_moving_eddies.py

@@ -146,7 +146,7 @@ def moving_eddies_example(
 
     # Execute for 1 week, with 1 hour timesteps and hourly output
     runtime = timedelta(days=7)
-    print("MovingEddies: Advecting %d particles for %s" % (npart, str(runtime)))
+    print(f"MovingEddies: Advecting {npart} particles for {runtime}")
     pset.execute(
         method,
         runtime=runtime,
@@ -177,7 +177,7 @@ def test_moving_eddies_fwdbwd(mode, mesh, tmpdir, npart=2):
     runtime = timedelta(days=1)
     dt = timedelta(minutes=5)
     outputdt = timedelta(hours=1)
-    print("MovingEddies: Advecting %d particles for %s" % (npart, str(runtime)))
+    print(f"MovingEddies: Advecting {npart} particles for {runtime}")
     outfile = tmpdir.join("EddyParticlefwd")
     pset.execute(
         method,

docs/examples/example_peninsula.py

@@ -168,7 +168,7 @@ def peninsula_example(
     out = (
         pset.ParticleFile(name=outfile, outputdt=timedelta(hours=1)) if output else None
     )
-    print("Peninsula: Advecting %d particles for %s" % (npart, str(time)))
+    print(f"Peninsula: Advecting {npart} particles for {time}")
     pset.execute(k_adv + k_p, runtime=time, dt=dt, output_file=out)
 
     if verbose:

docs/examples/example_stommel.py

@@ -160,7 +160,7 @@ def stommel_example(
 
     maxage = runtime.total_seconds() if maxage is None else maxage
     fieldset.add_constant("maxage", maxage)
-    print("Stommel: Advecting %d particles for %s" % (npart, runtime))
+    print(f"Stommel: Advecting {npart} particles for {runtime}")
     parcels.timer.psetinit.stop()
     parcels.timer.psetrun = parcels.timer.Timer("Pset_run", parent=parcels.timer.pset)
     pset.execute(

parcels/compilation/codegenerator.py

@@ -226,7 +226,7 @@ def __init__(self, fieldset=None, ptype=JITParticle):
 
     def get_tmp(self):
         """Create a new temporary variable name."""
-        tmp = "parcels_tmpvar%d" % self._tmp_counter
+        tmp = f"parcels_tmpvar{self._tmp_counter:d}"
         self._tmp_counter += 1
         self.tmp_vars += [tmp]
         return tmp

parcels/field.py

@@ -1242,7 +1242,7 @@ def _search_indices_curvilinear(self, x, y, z, ti=-1, time=-1, particle=None, se
             (xi, yi) = self._reconnect_bnd_indices(xi, yi, grid.xdim, grid.ydim, grid.mesh)
             it += 1
             if it > maxIterSearch:
-                print("Correct cell not found after %d iterations" % maxIterSearch)
+                print(f"Correct cell not found after {maxIterSearch} iterations")
                 raise FieldOutOfBoundError(x, y, 0, field=self)
         xsi = max(0.0, xsi)
         eta = max(0.0, eta)
@@ -2532,13 +2532,13 @@ def __init__(self, name: str, F, V=None, W=None):
                 assert isinstance(Fi, Field) and isinstance(
                     Vi, Field
                 ), "F, and V components of a NestedField must be Field"
-                self.append(VectorField(name + "_%d" % i, Fi, Vi))
+                self.append(VectorField(f"{name}_{i}", Fi, Vi))
         else:
             for i, Fi, Vi, Wi in zip(range(len(F)), F, V, W, strict=True):
                 assert (
                     isinstance(Fi, Field) and isinstance(Vi, Field) and isinstance(Wi, Field)
                 ), "F, V and W components of a NestedField must be Field"
-                self.append(VectorField(name + "_%d" % i, Fi, Vi, Wi))
+                self.append(VectorField(f"{name}_{i}", Fi, Vi, Wi))
         self.name = name
 
     def __getitem__(self, key):

parcels/kernel.py

@@ -418,7 +418,7 @@ def get_kernel_compile_files(self):
             dyn_dir = mpi_comm.bcast(dyn_dir, root=0)
             basename = cache_name if mpi_rank == 0 else None
             basename = mpi_comm.bcast(basename, root=0)
-            basename = basename + "_%d" % mpi_rank
+            basename = f"{basename}_{mpi_rank}"
         else:
             cache_name = (
                 self._cache_key

parcels/particle.py

@@ -201,7 +201,7 @@ def __del__(self):
 
     def __repr__(self):
         time_string = "not_yet_set" if self.time is None or np.isnan(self.time) else f"{self.time:f}"
-        p_string = "P[%d](lon=%f, lat=%f, depth=%f, " % (self.id, self.lon, self.lat, self.depth)
+        p_string = f"P[{self.id}](lon={self.lon:f}, lat={self.lat:f}, depth={self.depth:f}, "
         for var in vars(type(self)):
             if var in ["lon_nextloop", "lat_nextloop", "depth_nextloop", "time_nextloop"]:
                 continue

parcels/particledata.py

@@ -460,7 +460,7 @@ def getPType(self):
 
     def __repr__(self):
         time_string = "not_yet_set" if self.time is None or np.isnan(self.time) else f"{self.time:f}"
-        p_string = "P[%d](lon=%f, lat=%f, depth=%f, " % (self.id, self.lon, self.lat, self.depth)
+        p_string = f"P[{self.id}](lon={self.lon:f}, lat={self.lat:f}, depth={self.depth:f}, "
         for var in self._pcoll.ptype.variables:
             if var.name in [
                 "lon_nextloop",

parcels/particleset.py

@@ -1193,7 +1193,7 @@ def execute(
                             raise RuntimeError(
                                 "Field writing during execution only works for Fields with one snapshot in time"
                             )
-                        fldfilename = str(output_file.fname).replace(".zarr", "_%.4d" % fld.to_write)
+                        fldfilename = str(output_file.fname).replace(".zarr", f"_{fld.to_write:04d}")
                         fld.write(fldfilename)
                         fld.to_write += 1