-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add mart_gtfs.fct_vehicle_locations_grouped
#3660
Conversation
Warehouse report 📦 Checks/potential follow-upsChecks indicate the following action items may be necessary.
New models 🌱calitp_warehouse.mart.gtfs.fct_vehicle_locations_grouped DAGLegend (in order of precedence)
|
@vevetron: I see this note: |
I'm not sure, let me look into this. @charlie-costanzo might have an immediate answer. |
@tiffanychu90 I can't think of a reason we need to make new surrogate keys. If we are using this data, we either use vehicle_locations or vehicle_locations_grouped. I think the current approach, using the subset of keys is a good approach. The keys in fct_vehicle_positions are all unique, so this new table's keys should all be unique as well. |
Cool, then I think this PR is ready to merge. I ran the tests but it seemed ok, so now I'm not sure what was happening before where there was a proportion test not met. I guess we'll find out after merging! |
051a2cf
to
1522494
Compare
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.
LGTM
Please squash before merging. |
Description
Describe your changes and why you're making them. Please include the context, motivation, and relevant dependencies.
data-analyses/gtfs_funnel
work #3645mart_gtfs.fct_vehicle_locations_dwell
table #3646 with new imageType of change
How has this been tested?
Tests: No tests? Is this right?
Docs:
Post-merge follow-ups
Document any actions that must be taken post-merge to deploy or otherwise implement the changes in this PR (for example, running a full refresh of some incremental model in dbt). If these actions will take more than a few hours after the merge or if they will be completed by someone other than the PR author, please create a dedicated follow-up issue and link it here to track resolution.