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

Generate stations.json using the offical MTA station list CSV #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

georgyo
Copy link
Contributor

@georgyo georgyo commented Dec 1, 2022

The MTA does have listing of all stations in their dataset defined as "Station Locations" here

Every station is part of a Complex that can contain other stations. IE

Station ID,Complex ID,GTFS Stop ID,Division,Line,Stop Name,Borough,Daytime Routes,Structure,GTFS Latitude,GTFS Longitude,North Direction Label,South Direction Label,ADA,ADA Direction Notes,ADA NB,ADA SB,Capital Outage NB,Capital Outage SB
27,617,R31,BMT,4th Av,Atlantic Av-Barclays Ctr,Bk,D N R,Subway,40.683666,-73.97881,Manhattan,Coney Island - Bay Ridge,1,,,,,
40,617,D24,BMT,Broadway - Brighton,Atlantic Av-Barclays Ctr,Bk,B Q,Subway,40.68446,-73.97689,Manhattan,Brighton Beach & Coney Island,1,,,,,
133,133,L24,BMT,Canarsie,Atlantic Av,Bk,L,Elevated,40.675345,-73.903097,Manhattan,Canarsie - Rockaway Parkway,0,,,,,
338,617,235,IRT,Eastern Pky,Atlantic Av-Barclays Ctr,Bk,2 3 4 5,Subway,40.684359,-73.977666,Manhattan,Flatbush - New Lots,1,,,,,

Here you can see that Atlantic Av-Barclays Ctr has 3 rows, and the "Complex ID" is static accross all of them. This makes the station IDs consistent across all other MTA integrations as well.

http://web.mta.info/developers/developer-data-terms.html#data

@georgyo
Copy link
Contributor Author

georgyo commented Dec 3, 2022

This is likely not mergeable as is, because WTFT hard codes md5 generated station IDs for defaults, and people's stored favorites also use them. However getting to a consistent state of station IDs across MTA integrations would be useful.

@jonthornton
Copy link
Owner

Thanks for posting this! I agree with both of your points that it'd be an improvement but also a breaking change. I'll need to think about the migration path.

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.

2 participants