Remove extranious validations

Author: BrianMichell

src/mdio/ingestion/segy/coordinates.py

@@ -57,29 +57,13 @@ def _get_coordinates(
         segy_headers: Headers read in from SEG-Y file.
         mdio_template: The MDIO template to use for the conversion.
 
-    Raises:
-        ValueError: If a dimension or coordinate name from the MDIO template is not found in
-                    the SEG-Y headers.
-
     Returns:
         A tuple containing:
             - A list of dimension coordinates (1-D arrays).
             - A dict of non-dimension coordinates (str: N-D arrays).
     """
-    dimensions_coords = []
-    for dim_name in mdio_template.dimension_names:
-        if dim_name not in grid.dim_names:
-            err = f"Dimension '{dim_name}' was not found in SEG-Y dimensions."
-            raise ValueError(err)
-        dimensions_coords.append(grid.select_dim(dim_name))
-
-    non_dim_coords: dict[str, SegyHeaderArray] = {}
-    for coord_name in mdio_template.coordinate_names:
-        if coord_name not in segy_headers.dtype.names:
-            err = f"Coordinate '{coord_name}' not found in SEG-Y dimensions."
-            raise ValueError(err)
-        non_dim_coords[coord_name] = np.array(segy_headers[coord_name])
-
+    dimensions_coords = [grid.select_dim(name) for name in mdio_template.dimension_names]
+    non_dim_coords = {name: np.array(segy_headers[name]) for name in mdio_template.coordinate_names}
     return dimensions_coords, non_dim_coords
 
 

tests/unit/ingestion/test_segy_coordinates.py

@@ -52,36 +52,6 @@ def test_returns_dims_and_coords_in_template_order(self) -> None:
         np.testing.assert_array_equal(non_dim["cdp_x"], cdp_x)
         np.testing.assert_array_equal(non_dim["cdp_y"], cdp_y)
 
-    def test_missing_dimension_raises(self) -> None:
-        """A template dim missing from the grid should raise ValueError."""
-        grid = make_grid(
-            [
-                ("inline", np.array([1, 2], dtype=np.int32)),
-                # Missing 'crossline'
-                ("time", np.array([0, 4], dtype=np.int32)),
-            ]
-        )
-        headers = make_header_array({"cdp_x": np.zeros(2, dtype=np.float64), "cdp_y": np.zeros(2, dtype=np.float64)})
-
-        template = Seismic3DPostStackTemplate(data_domain="time")
-
-        with pytest.raises(ValueError, match=r"Dimension 'crossline' was not found"):
-            _get_coordinates(grid, headers, template)
-
-    def test_missing_coordinate_field_raises(self) -> None:
-        """A template coord absent from SEG-Y headers should raise ValueError."""
-        inline = np.array([1, 2], dtype=np.int32)
-        crossline = np.array([10, 20], dtype=np.int32)
-        sample = np.array([0, 4], dtype=np.int32)
-        grid = make_grid([("inline", inline), ("crossline", crossline), ("time", sample)])
-        # Headers lack 'cdp_y'
-        headers = make_header_array({"cdp_x": np.zeros(4, dtype=np.float64)})
-
-        template = Seismic3DPostStackTemplate(data_domain="time")
-
-        with pytest.raises(ValueError, match=r"Coordinate 'cdp_y' not found"):
-            _get_coordinates(grid, headers, template)
-
 
 class TestPopulateCoordinates:
     """Tests for the ``_populate_coordinates`` wrapper.