Conversation
There was a problem hiding this comment.
Pull request overview
Updates the GTFS catalog with new/updated schedule and GTFS-RT source entries referenced by the “February 17 Realtime Feeds” batch (Metro Bilbao schedule refresh plus multiple new/updated realtime feeds).
Changes:
- Update Metro Bilbao schedule source ID/“latest” URL and redirect a deprecated Bizkaia entry to the new source.
- Add new GTFS-RT source JSONs for several agencies (Caltrain, Metro Bilbao, dBus, Ilévia, JSP Skopje, Motion, multiple Passio feeds, Everett Transit).
- Deprecate older Everett Transit and SEPTA GTFS-RT entries and add redirects to the replacement source IDs.
Reviewed changes
Copilot reviewed 37 out of 37 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| catalogs/sources/gtfs/schedule/es-metro-bilbao-gtfs-3052.json | Updates Metro Bilbao schedule source ID and “latest” URL. |
| catalogs/sources/gtfs/schedule/es-bizkaia-metro-bilbao-gtfs-2683.json | Redirects deprecated Bizkaia Metro Bilbao schedule entry to new source ID. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-vp-3074.json | Adds new Everett Transit GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-vp-1549.json | Deprecates old Everett VP source and redirects to 3074. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-tu-3073.json | Adds new Everett Transit GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-tu-1551.json | Deprecates old Everett TU source and redirects to 3073. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-sa-3072.json | Adds new Everett Transit GTFS-RT Service Alerts source. |
| catalogs/sources/gtfs/realtime/us-washington-everett-transit-gtfs-rt-sa-1550.json | Deprecates old Everett SA source and redirects to 3072. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-rail-gtfs-rt-vp-3060.json | Adds new SEPTA Rail GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-rail-gtfs-rt-vp-1567.json | Deprecates old SEPTA Rail VP source and redirects to 3060. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-rail-gtfs-rt-tu-3061.json | Adds new SEPTA Rail GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-rail-gtfs-rt-tu-1568.json | Deprecates old SEPTA Rail TU source and redirects to 3061. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-bus-gtfs-rt-vp-3063.json | Adds new SEPTA Bus GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-bus-gtfs-rt-vp-1569.json | Deprecates old SEPTA Bus VP source and redirects to 3063. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-bus-gtfs-rt-tu-3062.json | Adds new SEPTA Bus GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-pennsylvania-septa-bus-gtfs-rt-tu-1570.json | Deprecates old SEPTA Bus TU source and redirects to 3062. |
| catalogs/sources/gtfs/realtime/us-motion-gtfs-rt-tu-3071.json | Adds new Motion GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-maryland-frederick-country-transit-gtfs-rt-vp-3070.json | Adds new Frederick County TransIT GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-maryland-frederick-country-transit-gtfs-rt-tu-3069.json | Adds new Frederick County TransIT GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-maryland-cecil-transit-gtfs-rt-vp-3080.json | Adds new Cecil Transit GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-maryland-cecil-transit-gtfs-rt-tu-3079.json | Adds new Cecil Transit GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-maryland-cecil-transit-gtfs-rt-sa-3078.json | Adds new Cecil Transit GTFS-RT Service Alerts source. |
| catalogs/sources/gtfs/realtime/us-indiana-south-bend-gtfs-rt-vp-3077.json | Adds new South Bend Public Transportation Corporation GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/us-indiana-south-bend-gtfs-rt-tu-3076.json | Adds new South Bend Public Transportation Corporation GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/us-indiana-south-bend-gtfs-rt-sa-3075.json | Adds new South Bend Public Transportation Corporation GTFS-RT Service Alerts source. |
| catalogs/sources/gtfs/realtime/us-california-california-caltrain-gtfs-rt-vp-3053.json | Adds new Caltrain GTFS-RT Vehicle Positions source (API-key auth). |
| catalogs/sources/gtfs/realtime/us-california-california-caltrain-gtfs-rt-tu-3054.json | Adds new Caltrain GTFS-RT Trip Updates source (API-key auth). |
| catalogs/sources/gtfs/realtime/us-california-california-caltrain-gtfs-rt-sa-3055.json | Adds new Caltrain GTFS-RT Service Alerts source (API-key auth). |
| catalogs/sources/gtfs/realtime/mk-skopje-jsp-skopje-gtfs-rt-tu-3067.json | Adds new JSP Skopje GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/mk-skopje-jsp-skopje-gtfs-rt-sa-3068.json | Adds new JSP Skopje GTFS-RT Service Alerts source. |
| catalogs/sources/gtfs/realtime/fr-lille-ilevia-gtfs-rt-tu-3059.json | Adds new Ilévia GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/es-bilbao-metro-gtfs-rt-vp-3058.json | Adds new Metro Bilbao GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/es-bilbao-metro-gtfs-rt-tu-3057.json | Adds new Metro Bilbao GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/es-bilbao-metro-gtfs-rt-sa-3056.json | Adds new Metro Bilbao GTFS-RT Service Alerts source. |
| catalogs/sources/gtfs/realtime/es-basque-dbus-gtfs-rt-vp-3066.json | Adds new dBus GTFS-RT Vehicle Positions source. |
| catalogs/sources/gtfs/realtime/es-basque-dbus-gtfs-rt-tu-3065.json | Adds new dBus GTFS-RT Trip Updates source. |
| catalogs/sources/gtfs/realtime/es-basque-dbus-gtfs-rt-sa-3064.json | Adds new dBus GTFS-RT Service Alerts source. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "entity_type": [ | ||
| "vp" | ||
| ], | ||
| "provider": "Metro Bilbao", |
There was a problem hiding this comment.
The provider is set to "Metro Bilbao", but this feed is for dBus (Donostiabus) (see the referenced static GTFS source 2682). Update the provider field to match the actual agency to avoid mislabeling in downstream catalogs.
| "provider": "Metro Bilbao", | |
| "provider": "dBus (Donostiabus)", |
| "503" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Service/rtTripUpdates.pb", |
There was a problem hiding this comment.
The SEPTA Rail trip updates URL path looks incorrect: it points to the Service endpoint. Per the issue details, the updated trip updates endpoint should use the Trip path (…/Trip/rtTripUpdates.pb).
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Service/rtTripUpdates.pb", | |
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Trip/rtTripUpdates.pb", |
| "502" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Service/rtTripUpdates.pb", |
There was a problem hiding this comment.
The SEPTA Bus trip updates URL path looks incorrect: it points to the Service endpoint. Per the issue details, the updated trip updates endpoint should use the Trip path (…/Trip/rtTripUpdates.pb).
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Service/rtTripUpdates.pb", | |
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Trip/rtTripUpdates.pb", |
| "mdb_source_id": 3068, | ||
| "data_type": "gtfs-rt", | ||
| "entity_type": [ | ||
| "sa" | ||
| ], | ||
| "provider": "JSP Skopje", | ||
| "static_reference": [ | ||
| "3045" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/alerts.pb", | ||
| "authentication_type": 0 |
There was a problem hiding this comment.
This JSON file uses tab characters and inconsistent indentation. Please reformat with spaces to match the prevailing style in the catalogs, to reduce noisy diffs and avoid tooling issues that assume space-indentation.
| "mdb_source_id": 3068, | |
| "data_type": "gtfs-rt", | |
| "entity_type": [ | |
| "sa" | |
| ], | |
| "provider": "JSP Skopje", | |
| "static_reference": [ | |
| "3045" | |
| ], | |
| "urls": { | |
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/alerts.pb", | |
| "authentication_type": 0 | |
| "mdb_source_id": 3068, | |
| "data_type": "gtfs-rt", | |
| "entity_type": [ | |
| "sa" | |
| ], | |
| "provider": "JSP Skopje", | |
| "static_reference": [ | |
| "3045" | |
| ], | |
| "urls": { | |
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/alerts.pb", | |
| "authentication_type": 0 |
| ], | ||
| "provider": "Frederick County TransIT", | ||
| "is_official": "True", | ||
| "static_reference": [ |
There was a problem hiding this comment.
The file path says "frederick-country-transit", but the provider and referenced static GTFS source (2432) are "Frederick County TransIT". Consider renaming these realtime source files to use "county" for consistency and discoverability.
| "entity_type": [ | ||
| "tu" | ||
| ], | ||
| "provider": "Metro Bilbao", |
There was a problem hiding this comment.
The provider is set to "Metro Bilbao", but this feed is for dBus (Donostiabus) (see the referenced static GTFS source 2682). Update the provider field to match the actual agency to avoid mislabeling in downstream catalogs.
| "provider": "Metro Bilbao", | |
| "provider": "dBus (Donostiabus)", |
| "entity_type": [ | ||
| "sa" | ||
| ], | ||
| "provider": "Metro Bilbao", |
There was a problem hiding this comment.
The provider is set to "Metro Bilbao", but this feed is for dBus (Donostiabus) (see the referenced static GTFS source 2682). Update the provider field to match the actual agency to avoid mislabeling in downstream catalogs.
| "provider": "Metro Bilbao", | |
| "provider": "dBus", |
| "503" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Service/rtVehiclePositions.pb", |
There was a problem hiding this comment.
The SEPTA Rail vehicle positions URL path looks incorrect: it points to the Service endpoint (and uses rtVehiclePositions.pb). Per the issue details, the updated VP endpoint should use the Vehicle path and filename (rtVehiclePosition.pb).
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Service/rtVehiclePositions.pb", | |
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Vehicle/rtVehiclePosition.pb", |
| "502" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Service/rtVehiclePositions.pb", |
There was a problem hiding this comment.
The SEPTA Bus vehicle positions URL path looks incorrect: it points to the Service endpoint (and uses rtVehiclePositions.pb). Per the issue details, the updated VP endpoint should use the Vehicle path and filename (rtVehiclePosition.pb).
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Service/rtVehiclePositions.pb", | |
| "direct_download": "https://www3.septa.org/gtfsrt/septabus-pa-us/Vehicle/rtVehiclePosition.pb", |
| "mdb_source_id": 3067, | ||
| "data_type": "gtfs-rt", | ||
| "entity_type": [ | ||
| "tu" | ||
| ], | ||
| "provider": "JSP Skopje", | ||
| "static_reference": [ | ||
| "3045" | ||
| ], | ||
| "urls": { | ||
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/tripupdates.pb", | ||
| "authentication_type": 0 |
There was a problem hiding this comment.
This JSON file uses tab characters and inconsistent indentation. Please reformat with spaces to match the prevailing style in the catalogs, to reduce noisy diffs and avoid tooling issues that assume space-indentation.
| "mdb_source_id": 3067, | |
| "data_type": "gtfs-rt", | |
| "entity_type": [ | |
| "tu" | |
| ], | |
| "provider": "JSP Skopje", | |
| "static_reference": [ | |
| "3045" | |
| ], | |
| "urls": { | |
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/tripupdates.pb", | |
| "authentication_type": 0 | |
| "mdb_source_id": 3067, | |
| "data_type": "gtfs-rt", | |
| "entity_type": [ | |
| "tu" | |
| ], | |
| "provider": "JSP Skopje", | |
| "static_reference": [ | |
| "3045" | |
| ], | |
| "urls": { | |
| "direct_download": "https://www.modeshift.app/api/v1/9814b106-2afe-47c8-919b-bdec6a5e521e/transport/gtfsrt/tripupdates.pb", | |
| "authentication_type": 0 |
| ], | ||
| "urls": { | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Service/rtVehiclePositions.pb", | ||
| "direct_download": "https://www3.septa.org/gtfsrt/septarail-pa-us/Vehicle/rtVehiclePositions.pb", |
jcpitre
left a comment
There was a problem hiding this comment.
Since the changes are a bit unusual, we should check the result in the DB after this PR is processed.
#1253
#717
#1254
#1247
#1241
#1240
#1236
#1234
#1216
#1213
#1207
#1199
#1197