Skip to content
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

Add support to geocoder and data-prep for new ITN local/express ferry classes #26

Closed
mraross opened this issue Mar 3, 2017 · 8 comments

Comments

@mraross
Copy link

mraross commented Mar 3, 2017

BC Ferries promised to refine their ferry classes. When these new classes make their way into ITN, geocoder prep will need to made aware of them.

@cmhodgson
Copy link
Collaborator

Support was already added for FERRY_PASSENGER.

@cmhodgson cmhodgson assigned mraross and unassigned cmhodgson and gleeming Apr 21, 2017
@mraross
Copy link
Author

mraross commented Apr 21, 2017

At MOTI's request, BC Ferries is going to create new ferry classes this fiscal to distinguish local from express so that routers won't jump from one the other at common docks.

@mraross mraross assigned cmhodgson and gleeming and unassigned mraross Apr 21, 2017
@mraross mraross changed the title Add support for new ITN ferry classes Add support for new ITN local/express ferry classes Apr 21, 2017
@cmhodgson
Copy link
Collaborator

Why would we not change ferries at a dock? What does "support" mean in terms of the geocoder? Just add them to the accepted road classes? Ferries segments don't have addressing anyway, I think we just need to ignore them at some level.

@mraross
Copy link
Author

mraross commented Apr 21, 2017

I might not have the new typology right but from what I recall, OnRoute was having trouble routing with the current ferry classes and we would too if BC Ferries didn't add new ferry classes

@cmhodgson cmhodgson changed the title Add support for new ITN local/express ferry classes Add support to geocoder and data-processing for new ITN local/express ferry classes Jun 5, 2017
@mraross mraross changed the title Add support to geocoder and data-processing for new ITN local/express ferry classes Add support to geocoder and data-prep for new ITN local/express ferry classes Jun 6, 2017
@cmhodgson cmhodgson assigned mraross and unassigned cmhodgson and gleeming Feb 5, 2018
@mraross
Copy link
Author

mraross commented Jun 14, 2018

Issue moved to bcgov/ols-router bcgov/api-specs#21 via ZenHub

@mraross mraross closed this as completed Jun 14, 2018
@mraross mraross reopened this Jun 14, 2018
@mraross mraross removed their assignment Feb 3, 2019
@mraross mraross transferred this issue from bcgov/api-specs Aug 31, 2019
@mraross mraross added this to the Geocoder 4.1 milestone Aug 31, 2019
@cmhodgson
Copy link
Collaborator

Doesn't this belong in ols-router, if anywhere? I don't think that ITN added these yet.

@mraross
Copy link
Author

mraross commented Apr 24, 2020

You're right. BC Ferries promised to restructure the ferry routes so its a future thing which is why I took it out of this year's work. I thought it would apply to both Geocodable BC Maker and Routable BC Maker.

@mraross mraross removed their assignment Jul 3, 2020
@alixcote
Copy link

alixcote commented Mar 3, 2023

Closing for now.

@alixcote alixcote closed this as completed Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants