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

Config for 50m distance between stop location and shape ? #793

Open
wkulesza opened this issue May 13, 2022 · 5 comments
Open

Config for 50m distance between stop location and shape ? #793

wkulesza opened this issue May 13, 2022 · 5 comments
Assignees

Comments

@wkulesza
Copy link

Observed behavior (please include a screenshot if possible)

In some cases (i.e. bus terminal with platforms that are out-of-reach for normal cars), we would like to have shape be close to stop location but default setting says 50m is max. Can this be configured ?

obraz

@philip-cline
Copy link
Contributor

Hi @wkulesza, this is a best practices check (derived from here) and so can be ignored without serious issues arising. However, in a case like this we agree it would be reasonable to make the value configurable, especially given that the current canonical GTFS validator uses 100m rather than our 50m. We can take a look at adding a Project Setting for the max distance from stop to pattern shape.

@wkulesza
Copy link
Author

@philip-cline thanks for getting back so quickly. Yes, I would like you to take a look and see if this could be configured in the config files. We could then also try and see what else we could bring up to config files and offer a PR in future.

@miles-grant-ibigroup
Copy link
Contributor

I have checked and it is not currently configurable. Making the value configurable would be quite trivial. Here is the relevant line of code:

export const PATTERN_TO_STOP_DISTANCE_THRESHOLD_METERS = 50

We welcome new PRs!

@philip-cline
Copy link
Contributor

@wkulesza It's also important to note here that this is really an indication that the stop should be probably be separated into child and parent stop IDs within the GTFS specification. From our perspective we would be interested to see some cases where this should be made configurable rather than adopting a child and parent stop relationship (although complexity with transit agencies will often make this difficult)

@wkulesza
Copy link
Author

@philip-cline this stop is separated into stops and station since this is a bus terminal, but OBA doesn't support it fully yet. This bus terminal is located within the area of a railway station, which makes it even more exciting.

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

3 participants