Conversation
equal_shape_distance_diff_coordinates
|
❌ Invalid acceptance test. |
|
❌ Invalid acceptance test. |
|
I did a deep dive into the data reported during our last release (4.2) and used that as the source of truth to compare the original number of feeds that triggered this error vs. the revised list from these acceptance tests. This threshold reduces the number of feeds triggered by this error by 50% (174 -> 87). If there are persistent user concerns after this rule change, we can revisit the threshold by doing analysis on all notice occurrences with the updated rule and the difference in metres for each occurrence, and then doing a distribution graph to help determine a more lenient threshold. For this release iteration, 1.11m is sufficient. |
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Show resolved
Hide resolved
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Outdated
Show resolved
Hide resolved
|
❌ Invalid acceptance test. |
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Outdated
Show resolved
Hide resolved
|
❌ Invalid acceptance test. |
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Show resolved
Hide resolved
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Show resolved
Hide resolved
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/mobilitydata/gtfsvalidator/validator/ShapeIncreasingDistanceValidator.java
Outdated
Show resolved
Hide resolved
|
❌ Invalid acceptance test. |
|
❌ Invalid acceptance test. |
Summary:$1.11 m$ threshold to distinguish between $\ge 1.11m$ ) and $\lt 1.11 m$ ) levels. Given the expansion of the Mobility Database since these initial discussions, reevaluating this threshold with current data offers a chance to refine the rule's sensitivity.
Closes #1638
Closes #1678
Concerns have been raised about the
equal_shape_dist_diff_coordinateserror's sensitivity to minor coordinate discrepancies. Historical discussions proposed aERROR(WARNING(Expected behavior:
equal_shape_distance_diff_coordinates, triggering anERRORfor distancesequal_shape_distance_diff_coordinates_below_threshold, withWARNINGseverity for distancesAcceptance tests results:$1.11m$ , thus qualifying for both
The acceptance tests results show a 6% reduction (93 notices) in
ERRORnotices. Additionally, 166 newWARNINGnotices were generated. This is because certain datasets exhibited distances between coordinates that were both greater than and less thanWARNINGandERRORnotices.Please make sure these boxes are checked before submitting your pull request - thanks!
gradle testto make sure you didn't break anything