-
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
Simplify round-trip routes #5
Comments
Another possibility for the avoid repetition feature: right now, avoid repetition simply tries to avoid running on the same path twice. But there could be another path adjacent to the original one and it'd happily use that. Perhaps avoid repetition should set a distance buffer around the edges already visited and then avoid entering that buffer. |
Notes to self... Main cause of lines crossing / complexity in routes is when routing around green polygons combined with the avoid repetition option. Things to experiment with:
While investigating, disable trail and water routes (done), disable fallback, disable duplicate removal and removal of routes that are too short/long |
Some improvements made and released, but still lots to be done. Changes made:
Convex hull work looked promising. Modifying avoid repetition to look for nearby edges is very inefficient, need to find a better way of handling this. Putting back on hold for now. |
Some round-trip routes end up with many many waypoints shown, leading to overly complex routes that would be hard to follow.
Possible options:
Example: https://trailrouter.com/#wps=51.54804,-0.02541%7C51.63373,-0.00381%7C51.63708,0.00042%7C51.63673,0.00414%7C51.63728,0.00127%7C51.63970,0.00201%7C51.64132,0.01066%7C51.63755,0.01374%7C51.64005,0.02244%7C51.63499,0.02571%7C51.63406,0.03323%7C51.63124,0.03170%7C51.63105,0.03229%7C51.63105,0.03229%7C51.63312,0.03371%7C51.63177,0.03646%7C51.63818,0.04068%7C51.64314,0.03870%7C51.64637,0.04211%7C51.64317,0.03853%7C51.64807,0.03657%7C51.64265,0.03262%7C51.64884,0.03384%7C51.65028,0.02250%7C51.64281,0.01145%7C51.64344,0.00012%7C51.64517,0.00444%7C51.64412,-0.00090%7C51.64371,-0.00090%7C51.64350,-0.00554%7C51.64352,-0.00117%7C51.64335,-0.00221%7C51.64053,-0.00084%7C51.63613,-0.00665%7C51.63565,-0.00248%7C51.54804,-0.02541&ss=&rt=true&td=40000&aus=false&aus2=false&ah=false&ar=true&pga=true&im=false
The text was updated successfully, but these errors were encountered: