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

Add realtime_trip_updates for kolejedolnoslaskie #1360

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

Conversation

sethidden
Copy link
Contributor

@sethidden sethidden commented Dec 26, 2024

I requested for GTFS creds for KD in #1034 so now I'm on their mailing list.

I' got an e-mail saying that the /vehicles endpoint provided GTFS-RT VehiclePositions, but now they have a /feed endpoint where they provide both VehiclePositions and TripUpdates

They explicitly ask to switch from /vehicles to /feed

I requested for GTFS creds for KD in transitland#1034 so now I'm on their mailing list.

I've got an e-mail saying that the /vehicles endpoint provided GTFS-RT VehiclePositions, but now they have a /feed endpoint where they provide both VehiclePositions and TripUpdates

They explicitly ask to *switch* from /vehicles to /feed
@@ -22,7 +22,8 @@
"id": "f-u3h-koleje~dolnoslaskie~rt",
"spec": "gtfs-rt",
"urls": {
"realtime_vehicle_positions": "https://gtfs.i.kiedyprzyjedzie.pl/rt/279/vehicles"
"realtime_vehicle_positions": "https://gtfs.i.kiedyprzyjedzie.pl/rt/279/feed",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that both properties point to exactly the same URL, without any ?type=vehicle_positions. Theoretically this means that trip_updates are fetched when trying to get only vehicle_positions and you'll need to strip the excess data

I did it like that because I see that in

"realtime_vehicle_positions": "https://gtfs-rt.infra-hubup.fr/mrchsl/realtime",
"realtime_trip_updates": "https://gtfs-rt.infra-hubup.fr/mrchsl/realtime"
it's done the same way

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