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

Abidjan Creator #154

Merged
merged 13 commits into from
Feb 1, 2020
Merged

Abidjan Creator #154

merged 13 commits into from
Feb 1, 2020

Conversation

nlehuby
Copy link
Collaborator

@nlehuby nlehuby commented Dec 13, 2019

Here come the first version of the Abidjan (Ivory Coast) creator 🎉

It uses operator and operator:website tags from OSM relations to set up the GTFS agencies.

It generates a GTFS with a frequencies.txt file and uses the logic described in #146:

  • the opening_hours, interval and interval:conditional tags on OSM relations are used to set up the schedule data
  • the duration tag is used to interpolate the time of travel between stops of each trip

We will use it in Abidjan for a while (our data collection is still ongoing) and if everything works as expected, we will go on with the issues #146 and #118, and move this logic to the default creator.

Some tests comes along the creator.

@nlehuby
Copy link
Collaborator Author

nlehuby commented Dec 23, 2019

Hey ! any thought about this PR ?

Merry Christmas to all the bus mappers in the world 🎅 🚏 🗺️

@pantierra
Copy link
Contributor

Looks good! I didn't look too much into the custom trips creator. Are you finished? or are you still working on it?

Can you squash the commits to a single one? Maybe with the two of you, one with Co-authored-by:.

@nlehuby
Copy link
Collaborator Author

nlehuby commented Feb 1, 2020

Yes, we are finished here. You can use the "squash and merge" github button

@pantierra pantierra merged commit 0f6a8fd into grote:master Feb 1, 2020
@nlehuby nlehuby deleted the ci_abidjan branch June 4, 2021 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants