-
Notifications
You must be signed in to change notification settings - Fork 16
feat: LiveView for exploring daily schedules for different bus routes #2842
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
base: main
Are you sure you want to change the base?
Conversation
thecristen
left a comment
There was a problem hiding this 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"}, |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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!
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:
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.