Skip to content

Clarification of stop_times.stop_id - #258

Merged
timMillet merged 2 commits into
google:masterfrom
MobilityData:clarification-stoptimes-stopid
Jan 4, 2021
Merged

Clarification of stop_times.stop_id#258
timMillet merged 2 commits into
google:masterfrom
MobilityData:clarification-stoptimes-stopid

Conversation

@timMillet

Copy link
Copy Markdown
Contributor

Following demand in the issue #257:

This pull request clarifies the definition of the fields stop_times.stop_id.

  • Explicitly tells that the referenced locations must have stops.location_type values of 0 or empty.
  • Future-proof phrasing: it references only what must be done.

Explicitely tells that the referenced locations must have a `stops.location_type` value of `0` or empty.
Future-proof phrasing: it references only what must be done.
@timMillet timMillet added the GTFS Schedule Issues and Pull Requests that focus on GTFS Schedule label Dec 22, 2020
@timMillet
timMillet requested a review from barbeau December 22, 2020 18:44
@timMillet timMillet self-assigned this Dec 22, 2020
@google-cla google-cla Bot added the cla: yes label Dec 22, 2020
@timMillet timMillet linked an issue Dec 22, 2020 that may be closed by this pull request
@mmorang

mmorang commented Dec 22, 2020

Copy link
Copy Markdown

Thanks, @timMillet. This looks good to me and is consistent with my understanding of the specification.

@skinkie

skinkie commented Dec 22, 2020

Copy link
Copy Markdown
Contributor

I am impatient to vote. Go Go Go! (OpenGeo)

@timMillet

Copy link
Copy Markdown
Contributor Author

@skinkie I'd tend to think that this change is editorial only, and would not require a vote. Do you think this change is worth a vote on your side?

@skinkie

skinkie commented Jan 4, 2021

Copy link
Copy Markdown
Contributor

No, I think this just makes things more clear.

@timMillet

Copy link
Copy Markdown
Contributor Author

Ok, thanks. I'll merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GTFS Schedule Issues and Pull Requests that focus on GTFS Schedule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can stop_times reference stops with location_type = 3 or 4?

3 participants