Normalize compact OFMX runway bearings

Author: ogrun

src/ofmx2pgsql/parser.py

@@ -220,8 +220,8 @@ def iter_runway_ends(path: Path) -> Iterator[RunwayEnd]:
             runway_ofmx_id=rwy_uid.get("mid") if rwy_uid is not None else None,
             airport_ofmx_id=ahp_uid.get("mid") if ahp_uid is not None else None,
             designator=_text(rdn_uid, "txtDesig"),
-            true_bearing=_to_float(_text(elem, "valTrueBrg")),
-            mag_bearing=_to_float(_text(elem, "valMagBrg")),
+            true_bearing=_parse_bearing(_text(elem, "valTrueBrg")),
+            mag_bearing=_parse_bearing(_text(elem, "valMagBrg")),
             latitude=_parse_coordinate(_text(elem, "geoLat")),
             longitude=_parse_coordinate(_text(elem, "geoLong")),
         )
@@ -370,6 +370,23 @@ def _parse_coordinate(value: Optional[str]) -> Optional[float]:
     return parsed
 
 
+def _parse_bearing(value: Optional[str]) -> Optional[float]:
+    parsed = _to_float(value)
+    if parsed is None:
+        return None
+    if 0 <= parsed <= 360:
+        return parsed
+    if not value:
+        return parsed
+    raw = value.strip()
+    if "." in raw or not raw.isdigit():
+        return parsed
+    scaled = int(raw) / 1000
+    if 0 <= scaled <= 360:
+        return round(scaled, 6)
+    return parsed
+
+
 def _parse_gml_pos_list(value: Optional[str]) -> list[tuple[float, float]]:
     if not value:
         return []

tests/test_parser.py

@@ -44,6 +44,15 @@ def test_iter_airspace_shapes_reads_sample(self) -> None:
         self.assertIsNotNone(first.ofmx_id)
         self.assertGreater(len(first.positions), 3)
 
+    def test_parse_bearing_normalizes_compact_out_of_range_value(self) -> None:
+        self.assertEqual(parser._parse_bearing("232230"), 232.23)
+
+    def test_parse_bearing_keeps_normal_value(self) -> None:
+        self.assertEqual(parser._parse_bearing("052"), 52.0)
+
+    def test_parse_bearing_preserves_leading_zero_fractional_value(self) -> None:
+        self.assertEqual(parser._parse_bearing("006169"), 6.169)
+
 
 if __name__ == "__main__":
     unittest.main()