Refactor lambda functions with ruff for improved readability in GDP and HURDAT2 adapters; update version to 0.47.0 (#589)

Author: selipot

clouddrift/adapters/gdp/gdpsource.py

@@ -494,10 +494,14 @@ def _process_chunk(
         preremove_df_chunk,
         filters=[
             # Filter out year values that are in the future or predating the GDP program
-            lambda df: (df["posObsYear"] > datetime.datetime.now().year)
-            | (df["posObsYear"] < 0),
-            lambda df: (df["senObsYear"] > datetime.datetime.now().year)
-            | (df["senObsYear"] < 0),
+            lambda df: (
+                (df["posObsYear"] > datetime.datetime.now().year)
+                | (df["posObsYear"] < 0)
+            ),
+            lambda df: (
+                (df["senObsYear"] > datetime.datetime.now().year)
+                | (df["senObsYear"] < 0)
+            ),
             # Filter out month values that contain non-numeric characters
             lambda df: df["senObsMonth"].astype(np.str_).str.contains(r"[\D]"),
             lambda df: df["posObsMonth"].astype(np.str_).str.contains(r"[\D]"),

clouddrift/adapters/hurdat2.py

@@ -417,16 +417,16 @@ def _extract_track_data(datafile_path: str, convert: bool) -> list[TrackData]:
     track_data = list[TrackData]()
 
     is_html_line = lambda line: re.match(r"[html|head|pre]+", line)
-    is_header_line = (
-        lambda cols, data_line_count: len(cols) == 4 and data_line_count == 0
+    is_header_line = lambda cols, data_line_count: (
+        len(cols) == 4 and data_line_count == 0
     )
     is_data_line = lambda cols, data_line_count: len(cols) == 21 and data_line_count > 0
     if convert:
-        nm_to_m = (
-            lambda x: float(x) * _METERS_IN_NAUTICAL_MILES
+        nm_to_m = lambda x: (
+            float(x) * _METERS_IN_NAUTICAL_MILES
         )  # nautical-miles to meters
-        k_to_mps = (
-            lambda x: float(x) * _METERS_IN_NAUTICAL_MILES / 3600
+        k_to_mps = lambda x: (
+            float(x) * _METERS_IN_NAUTICAL_MILES / 3600
         )  # knots to meters per second
         mb_to_pa = lambda x: float(x) * _PASCAL_PER_MILLIBAR  # millibar to pascal
     else:

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "clouddrift"
-version = "0.46.0"
+version = "0.47.0"
 
 authors = [
   { name="Shane Elipot", email="[email protected]" },

tests/ragged_test.py

@@ -705,8 +705,8 @@ def test_subset_by_rows(self):
         self.assertTrue(all(ds_sub["rowsize"] == [5, 4]))
 
     def test_subset_callable(self):
-        func = (
-            lambda arr: ((arr - arr[0]) % 2) == 0
+        func = lambda arr: (
+            ((arr - arr[0]) % 2) == 0
         )  # test keeping obs every two time intervals
         ds_sub = subset(self.ds, {"time": func})
         self.assertTrue(all(ds_sub["id"] == [1, 3, 2]))