Review feedback

Author: VeckoTheGecko

src/parcels/_core/fieldset.py

@@ -380,8 +380,7 @@ def from_sgrid_conventions(
         for varname in set(ds.data_vars) - set(fields.keys()) - skip_vars:
             fields[varname] = Field(varname, ds[varname], grid, XLinear)
 
-        fieldset = cls(list(fields.values()))
-        return fieldset
+        return cls(list(fields.values()))
 
 
 class CalendarError(Exception):  # TODO: Move to a parcels errors module

tests/test_advection.py

@@ -463,13 +463,7 @@ def test_nemo_curvilinear_fieldset():
     np.testing.assert_allclose(pset.lat, latp, atol=1e-1)
 
 
-@pytest.mark.parametrize(
-    "kernel",
-    [
-        AdvectionRK4,
-        AdvectionRK4_3D,
-    ],
-)
+@pytest.mark.parametrize("kernel", [AdvectionRK4, AdvectionRK4_3D])
 def test_nemo_3D_curvilinear_fieldset(kernel):
     data_folder = parcels.download_example_dataset("NemoNorthSeaORCA025-N006_data")
     U = xr.open_mfdataset(data_folder.glob("*U.nc"))