Skip to content

Feed Submission November 17th Part 2#1131

Merged
emmambd merged 7 commits intomainfrom
feat/november-17-feed-adds-2
Nov 18, 2025
Merged

Feed Submission November 17th Part 2#1131
emmambd merged 7 commits intomainfrom
feat/november-17-feed-adds-2

Conversation

@ianktc
Copy link
Copy Markdown
Contributor

@ianktc ianktc commented Nov 17, 2025

Address issues:

#1037: Add Scheduled Feed for Emerald Bay Shuttle Lake Tahoe
#1053: Add Scheduled Feed for West Cork Connect
#995: Add Scheduled Feed for Philadelphia PHILASH
#1043: Update Scheduled Feed for Tulare
#1040: Update Scheduled Feed for Butte

@ianktc
Copy link
Copy Markdown
Contributor Author

ianktc commented Nov 18, 2025

@emmambd Looks like there are some disallowed characters (! and ;) in the direct download URL. Should I reach to the provider to ask them to recreate a new URL name because this one is definitely weird:

https://urldefense.com/v3/__https:/data.trilliumtransit.com/gtfs/tcrta-ca-us/tcrta-ca-us.zip__;!!HOHAxFA!TXxjyCuVUPhvX7ywrcB1ByYGYjOqu4cggX9buu03x0mYoDLxdbEiAKsnuOwUjVsiD7Z7ATxq3U3NGNUepKMbCP-7LDWn9Q$

This is issue #1043

@davidgamez
Copy link
Copy Markdown
Member

@emmambd Looks like there are some disallowed characters (! and ;) in the direct download URL. Should I reach to the provider to ask them to recreate a new URL name because this one is definitely weird:

https://urldefense.com/v3/__https:/data.trilliumtransit.com/gtfs/tcrta-ca-us/tcrta-ca-us.zip__;!!HOHAxFA!TXxjyCuVUPhvX7ywrcB1ByYGYjOqu4cggX9buu03x0mYoDLxdbEiAKsnuOwUjVsiD7Z7ATxq3U3NGNUepKMbCP-7LDWn9Q$

This is issue #1043

Hi @ianktc, can you check this URL => https://data.trilliumtransit.com/gtfs/tcrta-ca-us/tcrta-ca-us.zip

Copy link
Copy Markdown
Contributor

@emmambd emmambd left a comment

Choose a reason for hiding this comment

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

Added some comments for things that need some additional investigation.

},
"subdivision_name": "Munster",
"municipality": "County Cork"
},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It might be worth seeing if this data has been made available via the Irish NAP: https://developer.nationaltransport.ie/

If not, it would be good for us to reach out to them about it once this PR is submitted to let them know.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In that case I would still add this as a separate feed then

"extracted_on": "2025-11-17 18:21:16.031697+00:00"
},
"subdivision_name": "Lake Tahoe",
"municipality": "Tahoe Regional Planning Agency"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Municipality looks wrong here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh yeah I thought so too but the spreadsheet has this under the municipality column. I could change it to something like Lake Tahoe though?
image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, sounds good

},
"urls": {
"direct_download": "https://data.trilliumtransit.com/gtfs/tcat-flex-ca-us/tcat-ca-us.zip",
"direct_download": "https://urldefense.com/v3/__https:/data.trilliumtransit.com/gtfs/tcrta-ca-us/tcrta-ca-us.zip__;!!HOHAxFA!TXxjyCuVUPhvX7ywrcB1ByYGYjOqu4cggX9buu03x0mYoDLxdbEiAKsnuOwUjVsiD7Z7ATxq3U3NGNUepKMbCP-7LDWn9Q$",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As David suggested, I'd recommend stripping the URL defense from the URL and retrying with the general URL

},
"urls": {
"direct_download": "https://www.blinetransit.com/documents/Google%20Transit/google_transit.zip",
"direct_download": "https://d16k74nzx9emoe.cloudfront.net/c4326823-2c12-4f59-82f1-3fc4d987e12b/google_transit.zip",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We have a National Transit Database feed that this feed should be redirecting to: https://mobilitydatabase.org/feeds/gtfs/ntd-90208

I'd compare the new URL to the URL for the ntd feed to see if they're different or not.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep they're the same! Redirecting to ntd-90208

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Set the status to deprecated and you're set :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ahh right!

"extracted_on": "2025-11-17 18:21:16.031697+00:00"
},
"subdivision_name": "Lake Tahoe",
"municipality": "Lake Tahoe"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed municipality to Lake Tahoe

@emmambd emmambd merged commit c7d7b56 into main Nov 18, 2025
10 checks passed
@emmambd emmambd deleted the feat/november-17-feed-adds-2 branch November 18, 2025 16:26
applecuckoo pushed a commit to applecuckoo/mobility-database-catalogs that referenced this pull request Nov 18, 2025
* add: schedule feeds for West Cork (mdb-2875) and Lake Tahoe (mdb-2876); addresses issues 1053 & 1037

* update: Tulare direct download URL; addresses issue 1043

* update: Butte direct download URL; addresses issue 1040

* add: Philly PHLASH scheduled feed; addresses issue 995

* update: addressing review comments

* Fix URL for direct download in GTFS JSON

Change to https:// to abide by regex checking

* Mark B-Line GTFS data as deprecated
applecuckoo pushed a commit to applecuckoo/mobility-database-catalogs that referenced this pull request Nov 18, 2025
* add: schedule feeds for West Cork (mdb-2875) and Lake Tahoe (mdb-2876); addresses issues 1053 & 1037

* update: Tulare direct download URL; addresses issue 1043

* update: Butte direct download URL; addresses issue 1040

* add: Philly PHLASH scheduled feed; addresses issue 995

* update: addressing review comments

* Fix URL for direct download in GTFS JSON

Change to https:// to abide by regex checking

* Mark B-Line GTFS data as deprecated
@ianktc ianktc self-assigned this Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants