Remove filtering subway predictions by route #663
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of changes
Asana Ticket: Investigate removing route-based filtering for predictions
This logic uses the
routes
fields in the source config for a stop to filter out predictions. This has caused a few instances, most recently for the new GLX stops, where trains that are being predicted in the feed do not show up on countdown clocks because the prediction is technically for a different route. A more specific example is an occasional Green Line B, C, or D train going up to Medford Tufts but not showing up on the countdown clocks because those signs source configs only hadGreen-E
in theroutes
field.In cases where there are multiple routes serving a station, there are always separate platform stop_ids that are used for each route (or route(s) for Green Line). JFK/UMass could be a potential special case but again, all platforms at that station have distinct platform stop_ids. Therefore, it should be safe and preferable to remove this filtering logic.
In addition, the historical context of this logic is that it was added to handle splitting out B and C/D predictions at Kenmore when the platform stop_ids were shared. Now we have distinct ones for the Kenmore B platforms so that further supports that we can safely remove the filtering logic #166
Testing done: