Feature/sedm clean by ryanaguilar · Pull Request #244 · edanalytics/edu_wh

ryanaguilar · 2026-05-15T15:43:42Z

feature/sedm: Add SEDM warehouse facts for student IEPs, IDEA events, and IEP disability lines

Description & motivation

This PR adds warehouse dbt models for the SEDM (Special Education Data Model) extension so analysts can use SEDM IEP and IDEA event data alongside existing core special-ed facts. Staging for these resources lives in edu_edfi_source (stg_sedm__student_ieps, stg_sedm__idea_events, stg_sedm__student_iep_disability_collections); this work materializes the corresponding facts in edu_wh under models/sedm_warehouse/, with inner joins to dim_student so k_student and k_student_xyear match the warehouse student dimension (same pattern as other student-centric facts).
Motivation: to support local education agencies' ability to support students with IEPs and facilitate mandated reporting requirements.

Breaking changes introduced by this PR:

This is a breaking change. The following sources and corresponding raw tables must be added:

  - name: idea_events
    description: >

    enabled: true
    columns: *column_defaults
  - name: student_ieps
    description: >

    enabled: true
    columns: *column_defaults
  - name: student_iep_disability_collections
    description: >

    enabled: true
    columns: *column_defaults

Models are intended to be gated by edu:sedm:enabled (default False) via dbt_project.yml once those configs are committed.

PR Merge Priority:

Low
Medium
High

Changes to existing files:

CHANGELOG entry under ## New Features.

New files created:

models/sedm_warehouse/fct_student_iep.sql: One row per student IEP from stg_sedm__student_ieps, keyed by k_student_iep. Resolves k_student / k_student_xyear from dim_student for referential integrity. Surrogate extension columns come from edu_edfi_source.extract_extension for stg_sedm__student_ieps. Post-hooks enforce PK on k_student_iep and NOT NULL on k_student_iep, k_student, k_student_xyear, plus an FK from k_student to dim_student.
models/sedm_warehouse/fct_idea_event.sql: One row per IDEA event from stg_sedm__idea_events, keyed by k_idea_event, with the same dim_student join and extension extraction pattern. Post-hooks: PK on k_idea_event, NOT NULL on key columns, FK to dim_student.
models/sedm_warehouse/fct_student_iep_disability.sql: Long-format rows for disabilities on an IEP from stg_sedm__student_iep_disability_collections, keyed by staging’s k_student_iep (and related descriptors / order). Post-hook adds an FK from k_student_iep to fct_student_iep.
Follow-ups (recommended before or right after merge, if not already on the branch):
Add sedm_warehouse folder config in dbt_project.yml (schema wh, enable via edu:sedm:enabled).

Tests and QC done:

dbt run --target data_dev successful in stadium_boston dev and stadium_south_carolina dev for all 3 new models

Submitter checklist

Make sure the following have been completed before approving this PR:

edu_wh PR Review Checklist:

Make sure the following have been completed before approving this PR:

ryanaguilar · 2026-05-18T13:19:12Z

      +schema: wh
+    sedm_warehouse:
+      +schema: wh
+      +enabled: "{{ var('edu:sedm:enabled', False) }}"


@rlittle08 I see that there is an existsing special education tag edu:special_ed. However, these new models are only available with the new SEDM Ed-Fi extension, so I'm not sure if I should use this new name and make a note about this distinction somewhere, or if these new models should fall under the existing variable.

rlittle08

this is in good shape, but left some naming suggestions, picky things, and an important q about the disability resource

rlittle08 · 2026-05-18T18:08:04Z

+        stage.tenant_code,
+        stage.school_year,
+        stage.idea_event_id,
+        stage.ed_org_id,


back in stage, can you add edorg_ref(), so we can get k_lea and k_school in the fct here?

rlittle08 · 2026-05-18T18:09:05Z

because student is part of the grain, can you rename here to fct_student_idea_event? base/stg can stay the same to follow practice of those models exactly matching the ed-fi resource

rlittle08 · 2026-05-18T18:10:04Z

+      tags: ['sedm']
+      enabled: "{{ var('edu:sedm:enabled', False) }}"
+
+    constraints:


thanks for including these!

rlittle08 · 2026-05-18T18:13:57Z

+        stage.idea_event_id,
+        stage.ed_org_id,
+        stage.ed_org_type,
+        stage.idea_event,


back in base/stg, I think we should rename this to idea_event_type, to follow our typical practice. There are exceptions, but i think this is a case where type is important info - this is capturing the type/category of event, right?

rlittle08 · 2026-05-18T18:16:10Z

+      - name: idea_event_id
+      - name: ed_org_id
+      - name: ed_org_type
+      - name: idea_event


can you fill in these idea-specific fields, using the data handbook descriptions?

rlittle08 · 2026-05-18T18:21:53Z

+        stage.special_education_hours_per_week
+        {{ edu_edfi_source.extract_extension(model_name='stg_sedm__student_ieps', flatten=False) }}
+    from stage
+        inner join dim_student


picky, we generally just do join not inner join, and don't indent this way

rlittle08 · 2026-05-18T18:22:21Z

+      - name: student_iep_association_id
+      - name: ed_org_id
+      - name: ed_org_type
+      - name: iep_finalized_date


can you please add descriptions to these by copying from ed-fi handbook?

rlittle08 · 2026-05-18T18:26:04Z

+        dim_student.k_student_xyear,
+        stage.tenant_code,
+        stage.school_year,
+        stage.student_iep_association_id,


see my comment on upstream pr about this col - the ed-fi docs call it by a different name?

rlittle08 · 2026-05-18T18:26:34Z

+}}
+
+with disabilities as (
+    select * from {{ ref('stg_sedm__student_iep_disability_collections') }}


see my comment on upstream pr - is this resource available in 6.1 data model? or has it moved to being a collection on studentIEP?

depending on your response, I may think about whether this should have its own fct model vs. be included in fct_student_iep

rlittle08 · 2026-05-18T18:31:16Z

i'm leaning toward a rename of fct_student_iep_association - thoughts?

ryanaguilar added 6 commits May 15, 2026 06:41

Add sedm models required for child count

cc69664

Update CHANGELOG.md

1072801

Add model ymls

b0caf49

Add schema and enabled var

6c3de2f

Add enable var to individual model ymls

9cb1321

Add breaking change flag to changelog

26ec0af

ryanaguilar marked this pull request as ready for review May 15, 2026 17:20

ryanaguilar requested a review from rlittle08 May 15, 2026 17:20

ryanaguilar commented May 18, 2026

View reviewed changes

rlittle08 requested changes May 18, 2026

View reviewed changes

ryanaguilar added 2 commits May 19, 2026 06:40

Add idea specific field descriptions

5016fe4

Add standard col descriptions

71f97b1

Conversation

ryanaguilar commented May 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

feature/sedm: Add SEDM warehouse facts for student IEPs, IDEA events, and IEP disability lines

Description & motivation

Breaking changes introduced by this PR:

PR Merge Priority:

Changes to existing files:

New files created:

Tests and QC done:

Submitter checklist

edu_wh PR Review Checklist:

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rlittle08 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ryanaguilar commented May 15, 2026 •

edited

Loading