Skip to content

Create Fact Tables for CTE, Migrant Education, Food Service Programs#175

Merged
rlittle08 merged 23 commits into
mainfrom
feature/program_wh_models
Sep 10, 2025
Merged

Create Fact Tables for CTE, Migrant Education, Food Service Programs#175
rlittle08 merged 23 commits into
mainfrom
feature/program_wh_models

Conversation

@mberrien-fitzsimons
Copy link
Copy Markdown
Contributor

feature/program_wh_models

Description & motivation

This PR creates fact tables for StudentMigrantEducationProgramAssociation, StudentCTEProgramAssociation, and StudentSchoolFoodServiceProgramAssociation.

PR Merge Priority:

  • Low
  • Medium
  • High

New files created:

  • fct_student_cte_program_associations : Creates fct table for StudentCTEProgramAssociation.
  • fct_student_migrant_education_program_associations : Creates fct table for StudentMigrantEducationProgramAssociation.
  • fct_student_school_food_service_program_associations : Creates fct table for StudentSchoolFoodServiceProgramAssociation.

Tests and QC done:

  • Executed a dbt project run and ensured it was successful.

@mberrien-fitzsimons
Copy link
Copy Markdown
Contributor Author

@sleblanc23 I've made updates to dim_students (and all newly associated models) as well as the bld_program_services models for CTE, food services, and migrant education. Please let me know if you have any questions.

Copy link
Copy Markdown
Contributor

@sleblanc23 sleblanc23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a few minor comments. If the primary key thing is actually wrong in those other models, could you please update their yaml files too?

Comment thread models/build/edfi_3/students/_edfi_3__students.yml Outdated
Comment thread models/core_warehouse/fct_student_cte_program_associations.sql Outdated
Comment thread models/core_warehouse/fct_student_cte_program_associations.yml Outdated
@mberrien-fitzsimons
Copy link
Copy Markdown
Contributor Author

@sleblanc23 , I've completed all comments.

Comment thread models/core_warehouse/fct_student_cte_program_associations.sql
Comment thread models/core_warehouse/fct_student_program_service.sql Outdated
Comment thread models/core_warehouse/fct_student_migrant_education_program_associations.sql Outdated
Comment thread models/build/edfi_3/students/_edfi_3__students.yml Outdated
Comment thread models/build/edfi_3/students/_edfi_3__students.yml
@mberrien-fitzsimons
Copy link
Copy Markdown
Contributor Author

I've worked through all comments and run DBT with variables enabled, disabled, and removed from dbt_project.yml file. Please let me know if there is anything else you need.

@rlittle08 rlittle08 added the 0.5.1 label Sep 9, 2025
@rlittle08 rlittle08 merged commit 2d4e048 into main Sep 10, 2025
@rlittle08 rlittle08 deleted the feature/program_wh_models branch September 10, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants