-
Notifications
You must be signed in to change notification settings - Fork 58
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-france: Use stable urls #481
base: main
Are you sure you want to change the base?
Conversation
It seems to be working, only 4 errors in the test run (missing agency.txt, all from the same feed producer) |
Will add these to the exclusion list then in the evening |
Unfortunately these feeds are fairly important, but return something like
Maybe the logic for picking the right resource needs some improvement. EDIT we are already picking the right resource, but "url" and "original_url" don't return the same. |
Yep indeed, SNCF feeds are relevant for most trains in France... I still don't see some data in transitous (Cap Cotentin related data notably) do you know why? |
Hi, feel free to read this very similar (deno) script https://github.com/laem/gtfs/blob/master/buildConfig.ts I had the same problem with some URLs returning index files instead of GTFS files... This comment : https://github.com/laem/gtfs/blob/master/buildConfig.ts#L157 I've informed the team about this. |
Thanks, it would be best if it was resolved upstream. Until then, regenerating the file often enough helps partially. |
No description provided.