-
Notifications
You must be signed in to change notification settings - Fork 6
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
segment speeds for speedmaps part 2 #1278
base: main
Are you sure you want to change the base?
Conversation
…quired per spec and can be nan. Instead, add in when joining to scheduled frequencies
nbviewer URLs for impacted notebooks: |
still pending (possibly in separate PRs):
|
Thanks! This all looks good to me. Approving this because my comment isn't a blocker, so it can be incorporated in future work. I recently added this into portfolio_utils to help generate / overwrite the portfolio yaml Small example of how it'd be used for you in
|
route_short_name
from grouping cols, instead re-add later (this was inadvertently dropping routes with only a long name)