Skip to content

Feature/epdm models evaluations#221

Draft
ryanaguilar wants to merge 11 commits into
mainfrom
feature/epdm_models__evaluations
Draft

Feature/epdm models evaluations#221
ryanaguilar wants to merge 11 commits into
mainfrom
feature/epdm_models__evaluations

Conversation

@ryanaguilar
Copy link
Copy Markdown
Contributor

@ryanaguilar ryanaguilar commented Mar 25, 2026

Description & motivation

Breaking changes introduced by this PR:

PR Merge Priority:

  • Low
  • Medium
  • High

Changes to existing files:

New files created:

Tests and QC done:

edu_wh PR Review Checklist:

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

  • Description of changes has been added to Unreleased section of CHANGELOG.md. Add under ## New Features for features, etc.
  • Code has been tested/checked for Databricks and Snowflake compatibility - EA engineers see Databricks checklist here
  • Reviewer confirms the grain of all tables are unchanged, OR any changes are expected, communicated, and this PR is flagged as a breaking change (not for patch release)
  • If a new configuration xwalk was added:
    • The code is written such that the xwalk is optional (preferred), and this behavior was tested, OR
    • The code is written such that the xwalk is required, and the required xwalk is added to edu_project_template, and this PR is flagged as breaking change (not for patch release)
    • A description for the new xwalk has been added to EDU documentation site here
  • If a new configuration variable was added:
    • The code is written such that the variable is optional (preferred), and this behavior was tested, OR
    • The code is written such that the variable is required, and a default value was added to edu_project_template, and this PR is flagged as breaking change (not for patch release)
    • A description for the new variable has been added to EDU documentation site here

ryanaguilar and others added 11 commits January 5, 2026 12:03
@ryanaguilar
Initial commit
cf063ed
@ryanaguilar
Fix typo
f53e82e
@ryanaguilar
Clear perf eval yml
316c526
@ryanaguilar
Add columns
eb26225
@ryanaguilar
Remove comment
d81e0da
@rlittle08 @ryanaguilar
add school year to fct models where it was missing (#215)
9b85f3c 
* add school year to fct models where it was missing

* update changelog
@gnguyen87 @ryanaguilar
Bugfix/class period (#214)
f370cc6 
* add logic to support non-military time records

* add doc to changelog

* add eval condition

---------

Co-authored-by: gnguyen87 <gnguyen@macalester.edu>
@rlittle08 @ryanaguilar
Bump/061 (#216)
730c52e 
* Update CHANGELOG.md

* Update dbt_project.yml
@ryanaguilar
Add dim_evaluation
9c49066
@ryanaguilar
Fix stg ref
a4ecda1
@ryanaguilar
Remove academic_subject, duplicate school_year
1fc5600
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ryanaguilar @rlittle08 @gnguyen87