feat: increase default horizon tolerance to 10%

Author: crorry-dev

CHANGELOG.md

@@ -18,9 +18,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Configurable Tolerance Buffer**: Reduce false positives from horizon calculations
   - `horizon_tolerance_percent` parameter in `validate()` and `validate_horizon()`
   - CLI option: `--horizon-tolerance FLOAT` (percentage above horizon line)
-  - **Default: 5.0%** - sensible default to account for typical calibration variations
+  - **Default: 10.0%** - sensible default to account for typical calibration variations
   - Annotations within tolerance buffer are considered valid
-  - Example: `--horizon-tolerance 10.0` accepts annotations up to 10% above calculated horizon
+  - Example: `--horizon-tolerance 15.0` accepts annotations up to 15% above calculated horizon
   - **Applies only to track and transition annotations** - other object types are not checked
   - Useful when horizon calculations are slightly inaccurate due to calibration variations
 

README.md

@@ -100,17 +100,17 @@ python -m raillabel_providerkit /path/to/folder_containing_scenes/ /path/to/outp
 For scenes where the horizon calculation might be slightly off due to calibration inaccuracies, you can add a tolerance buffer. This only applies to **track and transition** annotations:
 
 ```zsh
-# Add 5% tolerance above the calculated horizon line (default)
+# Use default 10% tolerance above the calculated horizon line
 python -m raillabel_providerkit validate /path/to/scenes/ /path/to/output/
 
 # Or customize the tolerance
-python -m raillabel_providerkit validate /path/to/scenes/ /path/to/output/ --horizon-tolerance 10.0
+python -m raillabel_providerkit validate /path/to/scenes/ /path/to/output/ --horizon-tolerance 15.0
 
 # With ontology validation
 python -m raillabel_providerkit validate /path/to/scenes/ /path/to/output/ --ontology config/parameters/osdar26.yaml --horizon-tolerance 10.0
 ```
 
-The tolerance is specified as a percentage. Annotations within this buffer zone above the horizon are considered valid. Default is 5.0% to account for typical calibration variations. This helps reduce false positives when horizon calculations are slightly inaccurate. **Note:** Horizon validation only checks track and transition annotations, not other object types.
+The tolerance is specified as a percentage. Annotations within this buffer zone above the horizon are considered valid. Default is 10.0% to account for typical calibration variations. This helps reduce false positives when horizon calculations are slightly inaccurate. **Note:** Horizon validation only checks track and transition annotations, not other object types.
 
 ## Exporting Scenes
 

raillabel_providerkit/__main__.py

@@ -272,10 +272,10 @@ def cli() -> None:
 @click.option(
     "--horizon-tolerance",
     type=float,
-    default=5.0,
+    default=10.0,
     help=(
-        "Tolerance buffer as percentage above horizon (e.g., 5.0 for 5%%). "
-        "Only applies to track/transition annotations. Default is 5.0."
+        "Tolerance buffer as percentage above horizon (e.g., 10.0 for 10%%). "
+        "Only applies to track/transition annotations. Default is 10.0."
     ),
 )
 @click.option("-q", "--quiet", is_flag=True, help="Disable progress bars")

raillabel_providerkit/validation/validate.py

@@ -42,7 +42,7 @@ def validate(  # noqa: C901, PLR0912, PLR0913
     validate_for_horizon: bool = True,
     validate_for_annotation_type_per_sensor: bool = True,
     validate_for_transition: bool = True,
-    horizon_tolerance_percent: float = 5.0,
+    horizon_tolerance_percent: float = 10.0,
 ) -> list[Issue]:
     """Validate a scene based on the Deutsche Bahn Requirements.
 
@@ -73,8 +73,8 @@ def validate(  # noqa: C901, PLR0912, PLR0913
         validate_for_transition: If True, issues are returned if transition annotations have
             identical startTrack and endTrack values. Default is True.
         horizon_tolerance_percent: Tolerance buffer as percentage above the horizon line.
-            Annotations within this buffer zone are considered valid. For example, 5.0 means
-            annotations up to 5% above the horizon line are accepted. Default is 5.0 (5% buffer).
+            Annotations within this buffer zone are considered valid. For example, 10.0 means
+            annotations up to 10% above the horizon line are accepted. Default is 10.0 (10% buffer).
             This only affects track and transition annotations.
 
 

raillabel_providerkit/validation/validate_horizon/validate_horizon.py

@@ -50,7 +50,7 @@ def _scene_uses_osdar26_calibration(scene: raillabel.Scene) -> bool:
 
 def validate_horizon(
     scene: raillabel.Scene,
-    horizon_tolerance_percent: float = 5.0,
+    horizon_tolerance_percent: float = 10.0,
 ) -> list[Issue]:
     """Validate whether all track/transition annotations are below the horizon.
 
@@ -67,8 +67,8 @@ def validate_horizon(
         Scene that should be validated.
     horizon_tolerance_percent : float, optional
         Tolerance buffer as percentage above the horizon line. Annotations within
-        this buffer zone are considered valid. For example, 5.0 means annotations
-        up to 5% above the horizon line are accepted. Default is 5.0 (5% buffer).
+        this buffer zone are considered valid. For example, 10.0 means annotations
+        up to 10% above the horizon line are accepted. Default is 10.0 (10% buffer).
         Helps reduce false positives from minor calibration inaccuracies.
 
     Returns