doppy 0.3.3__tar.gz → 0.3.4__tar.gz

{doppy-0.3.3 → doppy-0.3.4}/Cargo.lock

@@ -106,7 +106,7 @@ checksum = "248e3bacc7dc6baa3b21e405ee045c3047101a49145e7e9eca583ab4c2ca5345"
 
 [[package]]
 name = "doppy_rs"
-version = "0.3.3"
+version = "0.3.4"
 dependencies = [
  "doprs",
  "numpy",
@@ -115,7 +115,7 @@ dependencies = [
 
 [[package]]
 name = "doprs"
-version = "0.3.3"
+version = "0.3.4"
 dependencies = [
  "chrono",
  "rayon",

{doppy-0.3.3 → doppy-0.3.4}/Cargo.toml

@@ -4,6 +4,6 @@ resolver = "2"
 
 [workspace.package]
 edition = "2021"
-version = "0.3.3"
+version = "0.3.4"
 authors = ["Niko Leskinen <niko.leskinen@fmi.fi>"]
 license-file = "LICENSE"

{doppy-0.3.3 → doppy-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: doppy
-Version: 0.3.3
+Version: 0.3.4
 Classifier: Development Status :: 4 - Beta
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.10

{doppy-0.3.3 → doppy-0.3.4}/src/doppy/exceptions.py

@@ -8,3 +8,7 @@ class RawParsingError(DoppyException):
 
 class NoDataError(DoppyException):
     pass
+
+
+class ShapeError(DoppyException):
+    pass

{doppy-0.3.3 → doppy-0.3.4}/src/doppy/product/stare_depol.py

@@ -71,8 +71,6 @@ class StareDepol:
     radial_velocity: npt.NDArray[np.float64]
     mask: npt.NDArray[np.bool_]
     depolarisation: npt.NDArray[np.float64]
-    mask_depolarisation: npt.NDArray[np.bool_]
-    mask_beta_cross: npt.NDArray[np.bool_]
     polariser_bleed_through: float
     wavelength: float
     system_id: str
@@ -94,6 +92,12 @@ class StareDepol:
             The amount of bleed-through from the polariser.
         """
 
+        if co.beta.shape[1] != cross.beta.shape[1]:
+            raise doppy.exceptions.ShapeError(
+                "Range dimension mismatch in co and cross: "
+                f"{co.beta.shape[1]} vs {cross.beta.shape[1]}"
+            )
+
         if not np.isclose(co.wavelength, cross.wavelength):
             raise ValueError(
                 "Different wavelength in co and cross: "
@@ -107,12 +111,6 @@ class StareDepol:
         if not np.allclose(co.radial_distance, cross.radial_distance, atol=1):
             raise ValueError("Different radial distance in co and cross")
 
-        if co.beta.shape[1] != cross.beta.shape[1]:
-            raise ValueError(
-                "Range dimension mismatch in co and cross: "
-                f"{co.beta.shape[1]} vs {cross.beta.shape[1]}"
-            )
-
         ind = np.searchsorted(cross.time, co.time, side="left")
         pick_ind = ind < len(cross.time)
         time_diff_threshold = 2 * np.median(np.diff(co.time))
@@ -142,12 +140,18 @@ class StareDepol:
         self.radial_velocity = co.radial_velocity
         self.mask = co.mask
         self.depolarisation = depolarisation
-        self.mask_depolarisation = np.isnan(depolarisation)
-        self.mask_beta_cross = np.isnan(self.beta_cross)
         self.polariser_bleed_through = polariser_bleed_through
         self.wavelength = co.wavelength
         self.system_id = co.system_id
 
+    @property
+    def mask_depolarisation(self) -> npt.NDArray[np.bool_]:
+        return np.isnan(self.depolarisation)
+
+    @property
+    def mask_beta_cross(self) -> npt.NDArray[np.bool_]:
+        return np.isnan(self.beta_cross)
+
     @classmethod
     def from_halo_data(
         cls,

{doppy-0.3.3 → doppy-0.3.4}/src/doppy/product/wind.py

@@ -34,6 +34,18 @@ class Wind:
     system_id: str
     options: Options | None
 
+    @property
+    def mask_zonal_wind(self) -> npt.NDArray[np.bool_]:
+        return np.isnan(self.zonal_wind)
+
+    @property
+    def mask_meridional_wind(self) -> npt.NDArray[np.bool_]:
+        return np.isnan(self.meridional_wind)
+
+    @property
+    def mask_vertical_wind(self) -> npt.NDArray[np.bool_]:
+        return np.isnan(self.vertical_wind)
+
     @functools.cached_property
     def horizontal_wind_speed(self) -> npt.NDArray[np.float64]:
         return np.sqrt(self.zonal_wind**2 + self.meridional_wind**2)
@@ -153,7 +165,7 @@ class Wind:
         elevation = np.array(elevation_list)
         wind = np.concatenate(wind_list)
         rmse = np.concatenate(rmse_list)
-        mask = _compute_mask(wind, rmse)
+        mask = _compute_mask(wind, rmse) | np.any(np.isnan(wind), axis=2)
         if not np.allclose(elevation, elevation[0]):
             raise ValueError("Elevation is expected to stay same")
         height = np.array(raw.height, dtype=np.float64)
@@ -241,6 +253,7 @@ class Wind:
                 dimensions=("time", "height"),
                 units="m s-1",
                 data=self.zonal_wind,
+                mask=self.mask_zonal_wind,
                 dtype="f4",
                 long_name="Non-screened zonal wind",
             )
@@ -249,7 +262,7 @@ class Wind:
                 dimensions=("time", "height"),
                 units="m s-1",
                 data=self.zonal_wind,
-                mask=self.mask,
+                mask=self.mask | self.mask_zonal_wind,
                 dtype="f4",
                 long_name="Zonal wind",
             )
@@ -258,6 +271,7 @@ class Wind:
                 dimensions=("time", "height"),
                 units="m s-1",
                 data=self.meridional_wind,
+                mask=self.mask_meridional_wind,
                 dtype="f4",
                 long_name="Non-screened meridional wind",
             )
@@ -266,7 +280,7 @@ class Wind:
                 dimensions=("time", "height"),
                 units="m s-1",
                 data=self.meridional_wind,
-                mask=self.mask,
+                mask=self.mask | self.mask_meridional_wind,
                 dtype="f4",
                 long_name="Meridional wind",
             )

{doppy-0.3.3 → doppy-0.3.4}/src/doppy/raw/halo_bg.py

@@ -12,6 +12,8 @@ import numpy as np
 import numpy.typing as npt
 from numpy import datetime64
 
+from doppy.exceptions import RawParsingError
+
 
 @dataclass
 class HaloBg:
@@ -30,6 +32,28 @@ class HaloBg:
         | Sequence[tuple[bytes, str]]
         | Sequence[tuple[BufferedIOBase, str]],
     ) -> list[HaloBg]:
+        """
+        Creates a list of `HaloBg` instances from various data sources.
+
+        Parameters
+        ----------
+        data
+            A sequence of data source identifiers which can be file paths (as strings
+            or `Path` objects), tuples of raw byte data with filenames, or tuples of
+            buffered reader streams with filenames.
+
+        Returns
+        -------
+        list[HaloBg]
+            A list of `HaloBg` instances created from the provided data sources. Data
+            sources that cause a raw parsing error are ignored and not included in
+            the resulting list.
+
+        Raises
+        ------
+        TypeError
+            If `data` is not a list or tuple of supported types.
+        """
         if not isinstance(data, (list, tuple)):
             raise TypeError("data should be list or tuple")
         # TODO: rust reader and proper type checking
@@ -46,10 +70,13 @@ class HaloBg:
                 data_normalised.append(item)
             elif isinstance(item, tuple) and isinstance(item[0], BufferedIOBase):
                 data_normalised.append((item[0].read(), item[1]))
-        return [
-            HaloBg.from_src(data_bytes, filename)
-            for data_bytes, filename in data_normalised
-        ]
+        bgs = []
+        for data_bytes, filename in data_normalised:
+            try:
+                bgs.append(HaloBg.from_src(data_bytes, filename))
+            except RawParsingError:
+                continue
+        return bgs
 
     @classmethod
     def from_src(
@@ -124,9 +151,13 @@ def _from_src(data: BufferedIOBase, filename: str) -> HaloBg:
         try:
             signal = np.array(list(map(float, data_bytes.split(b"\r\n"))))[np.newaxis]
         except ValueError:
-            signal = np.array(
-                list(map(float, data_bytes.replace(b",", b".").split(b"\r\n")))
-            )[np.newaxis]
+            try:
+                signal = np.array(
+                    list(map(float, data_bytes.replace(b",", b".").split(b"\r\n")))
+                )[np.newaxis]
+            except ValueError as err:
+                raise RawParsingError(err) from err
+
     return HaloBg(time, signal)
 
 

{doppy-0.3.3 → doppy-0.3.4}/src/doppy/raw/windcube.py

@@ -216,7 +216,7 @@ def _extract_float64_or_raise(nc: Dataset) -> npt.NDArray[np.float64]:
             if nc.units != "dB":
                 raise ValueError(f"Unexpected units for {nc.name}")
             if nc[:].mask is not np.bool_(False):
-                raise ValueError
+                pass  # ignore that array contains masked values
             return np.array(nc[:].data, dtype=np.float64)
         case "radial_wind_speed":
             if nc.dimensions != ("time", "gate_index"):
@@ -224,7 +224,7 @@ def _extract_float64_or_raise(nc: Dataset) -> npt.NDArray[np.float64]:
             if nc.units != "m s-1":
                 raise ValueError(f"Unexpected units for {nc.name}")
             if nc[:].mask is not np.bool_(False):
-                raise ValueError
+                pass  # ignore that array contains masked values
             return np.array(nc[:].data, dtype=np.float64)
         case "radial_wind_speed_ci":
             if nc.dimensions != ("time", "gate_index"):
@@ -232,7 +232,7 @@ def _extract_float64_or_raise(nc: Dataset) -> npt.NDArray[np.float64]:
             if nc.units != "percent":
                 raise ValueError(f"Unexpected units for {nc.name}")
             if nc[:].mask is not np.bool_(False):
-                raise ValueError
+                pass  # ignore that array contains masked values
             return np.array(nc[:].data, dtype=np.float64)
         case "azimuth" | "elevation":
             if nc.dimensions != ("time",):