Skip to content

SF2.0[fix]: Skipped Stops disappear when the schedule drifts into the past, even if the vehicle is still upstream#3230

Open
lvachon1 wants to merge 3 commits into
mainfrom
lev/fix/sf2.0/past_skipped_stops
Open

SF2.0[fix]: Skipped Stops disappear when the schedule drifts into the past, even if the vehicle is still upstream#3230
lvachon1 wants to merge 3 commits into
mainfrom
lev/fix/sf2.0/past_skipped_stops

Conversation

@lvachon1
Copy link
Copy Markdown
Contributor

@lvachon1 lvachon1 commented Jun 4, 2026

Scope

Asana Ticket: 📅🔎 Skipped Stops disappear when the schedule drifts into the past, even if the vehicle is still upstream

Implementation

Removed time filtering from predicted schedules on trip details so that we continue to show skipped stops that have predictions in the past, but the vehicle hasn't skipped yet on this trip

Screenshots

Local ------------------------- Dev

Screenshot 2026-06-04 at 1 40 21 PM

How to test

Find a route with skipped stops, preferably a bus. Find a stop right before the skipped stops, watch a vehicle approach and confirm that the skipped stops do not disappear while the vehicle is still on it's way to your stop.

If there's a Red Sox home game, the 8, 19, 60, and 65 will be skipping stops around Fenway and you can use this link and vary the route between those to find a bus that is coming soon. I got lucky, these seem to be rather infrequent busses.

https://dev.mbtace.com/departures/?route_id=65&direction_id=1&stop_id=1806

…at we continue to show skipped stops that have predictions in the past, but the vehicle hasn't skipped yet on this trip
@lvachon1 lvachon1 requested a review from a team as a code owner June 4, 2026 17:45
@lvachon1 lvachon1 requested a review from joshlarson June 4, 2026 17:45
Copy link
Copy Markdown
Contributor

@joshlarson joshlarson left a comment

Choose a reason for hiding this comment

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

I do think this change deserves a test in UpcomingDeparturesTest.

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