-
Notifications
You must be signed in to change notification settings - Fork 13
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
New NTD mart table dim_annual_funding_sources #3495
Conversation
c5a36ab
to
fd9b33e
Compare
Warehouse report 📦 Checks/potential follow-upsChecks indicate the following action items may be necessary.
New models 🌱calitp_warehouse.mart.ntd.dim_annual_funding_sources DAGLegend (in order of precedence)
|
fd9b33e
to
4711544
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.
Can the table be renamed to dim_annual_agency_funding_source
?
Thoughts on skipping the ntd
portion since this is within mart_ntd
already? Since this is one of the first tables, probably the convention can somewhat follow [dim/fct]_[time grain: annual/monthly]_[grain: agency, agency-mode]_[some other descriptor]
?
Makes totally sense! Will rename it .
|
4711544
to
4099d85
Compare
…l records from federal, state, and local funding source tables. [#3405]
4099d85
to
e578ead
Compare
Description
This new model
dim_annual_funding_sources
details federal, state, and local funding sources for each applicable agency reporting to the NTD in Report Year 2022.[#3405]
Type of change
How has this been tested?
Tested locally using command:
poetry run dbt run --select "models/mart/ntd/dim_annual_funding_sources.sql"
Model was generated on staging:
cal-itp-data-infra-staging.erika_mart_ntd.dim_annual_funding_sources
Post-merge follow-ups
Confirm the creation of
mart_ntd.dim_annual_funding_sources
oncal-itp-data-infra
after the nexttransform_warehouse
runs.