-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restrict SG projection during SG matching on directly connected route segments #69
Comments
We discussed it today. We think it's an edge case that doesn't occur very often (based on recorded tracks). We consider the following to be the most robust solution that doesnt affect the previous matching: Split the route at intersections (where the route intersects) into multiple route segments and perform individual SG matching for these segments. This leaves normal tracks without intersections unaffected. For the intersection edge cases, however, we would be protected against projections of SGs onto different route segments. Although it is possible that false negatives could occur at the ends of the individual route segments (if an SG Map goes beyond an end), we still consider this to be the best solution. |
Implementing the discussed concept is not as trivial as assumed (GeoDjango and Shapely do not have native APIs for the splitting of a Linestring at own intersection points). We will therefore postpone this for the time being and work on other things. The current progress is on the following branch: https://github.com/priobike/priobike-sg-selector/tree/feature/robustness-against-route-intersections |
See #79 for a possible solution and explanation of that solution |
During our test rides there were occasions where routes where crossing itself (can be caused by round trips). On such intersections currently the SG projection (part of SG matching) may project the SG on two different route segments. This may result in an erroneous matching. The SG should only be projected on continuous route segments.
The text was updated successfully, but these errors were encountered: