Skip to content

Conversation

@joshlarson
Copy link
Contributor

@joshlarson joshlarson commented Dec 15, 2025

No ticket, but we've been doing a lot of thinking about services, and how they interact to form daily schedules.

This creates a preview page that can be used to explore which services are active for a given route on the same days, thus dividing the days in a rating up into different days with the exact same trips (same by trip ID, not same by which stops are visited when).

See the example below:

Screenshot 2025-12-15 at 5 09 35 PM

This isn't intended for riders - it's just another way to examine schedules - maybe we can clean this up and combine "daily schedules" into something that we do want to show riders.

@joshlarson joshlarson requested a review from a team as a code owner December 15, 2025 22:15
Copy link
Collaborator

@thecristen thecristen left a comment

Choose a reason for hiding this comment

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

To use internally, can you add some comments into the code itself? You have a nice description in the PR but someone coming cold to this code is not going to have any idea what's going on. 😅

date_set = schedule.dates |> MapSet.new()

[
{fn -> MapSet.subset?(date_set, MapSet.new([~D[2025-11-27]])) end, "Thanksgiving"},
Copy link
Collaborator

Choose a reason for hiding this comment

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

For the holidays, I'm curious why you didn't want to use the added_date_notes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hummm - I don't remember, tbh!

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.

3 participants