feat(pbf-patch): open footpaths around targets without road access - #4
Merged
Conversation
choffmann
force-pushed
the
feat/allowed-paths
branch
from
August 22, 2026 11:45
ec0f11a to
53a9ab4
Compare
- pin which radius drives which check: a test with asymmetric parameters now fails when road_max_distance and footpath_search_radius are swapped, a mutation the previous suite let through - report the feature index for real non-Point geometries, and reject a non-numeric coordinate instead of silently skipping it - exclude vehicle=no, which mappers use for paths a vehicle cannot pass - drop the unreachable Steps exclusion: a footpath candidate can never carry highway=steps - report targets left without any footpath in range, with their ids - keep an existing maxspeed or maxweight instead of loosening it - drop the candidate cap in the radius queries: the R-tree ranks by bounding box, so a capped list can hide a closer way
choffmann
force-pushed
the
feat/allowed-paths
branch
from
August 22, 2026 11:58
53a9ab4 to
ffe2f0f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.