Open mvl22 opened 8 years ago
Please can you put this in our main issue tracker (but only if it affects the PCT): https://github.com/npct/pct-shiny/issues - I would suggest we use a systematic method to move those 'unroutables'! Many thanks for finding them.
The following locations are not routable, due to them being in the midst of non-cycleable areas which are technically routable locally by bicycle but which can't be exited from, e.g. motorways or one-way systems.
I recommend these be manually adjusted, as routes from/to these specific locations will fail.
I used the script created by @Robinlovelace in 4a962678cd1e38d662929fd4541c20674d292d34 and ran this through our batch routing system, which picks up these cases quickly.
This is not at this stage a complete list as I tried the batch on a server that only has about half of the UK loaded. I will edit this opening posting if more are found in future.
(It's on our list to fix these rare cases at engine level, but it's non-trivial because the local routability remains valid. Essentially we need a re-entrant algorithm that can invalidate a nearest point lookup if the other end is on a different network island and then second time around treat the local-only network as if it were bicycle=no even though it the data is bicycle=yes. I have a feeling that Simon has been doing some work on this general area recently but we don't have a stable engine implementation yet.)