bcgov / ols-router

BC Advanced Route Planner
https://bcgov.github.io/ols-router/
Apache License 2.0
23 stars 11 forks source link

In ACDF, a route from anywhere to Saanich Peninsula Hospital results in route not found #278

Closed mraross closed 3 years ago

mraross commented 4 years ago

In ACDF, a route from anywhere to Saanich Peninsula Hospital at 2166 MOUNT NEWTON CROSS ROAD, SAANICHTON results in a Route not found error.

We did a test in betweenPairs demo app which confirmed correct operation of Route Planner as follows:

image.png

mraross commented 4 years ago

Brian geocoded 2166 Mt Newton Cross Rd to get a parcelPoint which he then fed into a custom betweenPairs request and it failed, giving a Route no found error. He then looked at a map of the parcelPoint as follows:

image

and realized the parcel is very big and the parcelPoint was likely beyond snapping tolerance to Mt Newton Cross Rd . He proposed that the issue resolution is simply to use the accessPoint of the hospital address (48.5941505, -123.407082), not the parcelPoint in ACDF

NicoledeGreef commented 4 years ago

imapbc puts 2166 mount newton cross road here: image but the hospital with that address is actually the pink parcel (Crown Provincial Ownership) whereas the yellow parcels are Privately owned. the parcels have different PIDs. i am confused...

mraross commented 4 years ago

We just take the parcelPoint the muni (Central Saanich) gives us. In any case, the issue resolution is the same because the parcel centroid of the correct parcel is also beyond snapping distance of the road.

NicoledeGreef commented 4 years ago

the ACDF addresses are being run through the ALE once a month before re-loading. what does the address list editor use - accesspoint as opposed to parcelpoint? there is a need to be consistent, i would think.

looking at the muni's map they have those two parcels under the same address; image

mraross commented 4 years ago

ALE returns parcelPoints. You could use the batch geocoder. It allows you to specify a locationDescriptor of accessPoint. Saanich Peninsula Hospital is probably the only location in your list on that big a parcel so you could just plug the access point coordinates from this ticket (see above) into your csv file after you've processed it in the ALE. But, to be safe, and avoid confusion, batch geocoder is better.

NicoledeGreef commented 4 years ago

maybe that's the issue then. i'll try the batch geocoder and see if it fixes the issue.

NicoledeGreef commented 4 years ago

accessPoint (red dot)versus parcelPoint (pin) as geocoded by address search in imapBC:

image

Testing the data once it's loaded to DLV and TST tomorrow.

mraross commented 4 years ago

Red dot looks good for routing!

NicoledeGreef commented 4 years ago

locationDescriptor of "accessPoint" when geocoding has fixed the Saan Pen Hospital issue the end users were seeing in ACDF. thanks for the assistance in tracking down the cause of the issue.

mraross commented 4 years ago

Woo hoo!!

mraross commented 4 years ago

Verified in production