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

Added routes() function to create MultiLineString geojson #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AvidDabbler
Copy link

I was looking for a good solution for simply and performantly creating routes MultiLineString geojson and I came across your repo.

I added a function called routes() that finds all of the trips in a route, gather all of the shape_id's and loops through each of the shapes to see if it needs to be added to the route MultiLineString geojson file.

I tried to follow the pattern that you all set by loading the csv file prior, then parsing in the function. The routes function does need 3 different files in order to work though (routes, trips, and shapes).

I updated the routes.input file and added the trips and shapes files in the testing folder and added the routes() to the testing script.

Seems to all be passing. :D

@AvidDabbler
Copy link
Author

if you need me to make the input and geojson files smaller in test I can try to take a look at that this week

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.

1 participant