Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Aggregating vp_usable trips to routes #1040

Merged
merged 7 commits into from
Feb 29, 2024
Merged

Aggregating vp_usable trips to routes #1040

merged 7 commits into from
Feb 29, 2024

Conversation

amandaha8
Copy link
Contributor

@amandaha8 amandaha8 commented Feb 29, 2024 •
edited by tiffanychu90
Loading

  • Updated work per Tiffany's feedback.
  • Feedback 1: updated the grouping columns for aggregating trips up to routes so it would be less granular.
  • Feedback 2: adjusted the routes aggregation function so the routes are rolled up by peak/offpeak. Added the time_period column with values that match with other datasets such as this and this.
  • Double checked that the routes_df merged nicely with outputs here
  • Begin troubleshooting why San Francisco Muni is totally missing?

@amandaha8 amandaha8 merged commit 0506c29 into main Feb 29, 2024
3 checks passed
@amandaha8 amandaha8 deleted the ah_routes branch February 29, 2024 22:45
@github-actions GitHub Actions
Copy link

nbviewer URLs for impacted notebooks:

@tiffanychu90
Copy link
Member

@github-actions GitHub Actions
Copy link

nbviewer URLs for impacted notebooks:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amandaha8 @tiffanychu90