Skip to content

Conversation

@ryanaguilar
Copy link
Contributor

Description & motivation

dim_candidate model for EPDM data. Created as part of summer intern final project and split from original PR that included multiple models. Jira task: STAD-213

PR Merge Priority:

  • Medium

New files created:

  • dim_candidate.sql: dimension table to describe an educator candidate
  • dim_candidate.yml: YAML to describe table above

Tests and QC done:

  • Executed successful dbt project run in TX stadium.

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.
  • 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
  • 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)
  • Build the other models related to this project.

@ryanaguilar ryanaguilar requested a review from rlittle08 October 21, 2025 17:31
k_candidate,
k_person,
tenant_code,
api_year,
Copy link
Collaborator

Choose a reason for hiding this comment

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

we tend not to include api_year in wh models, bc unlike school_year, it doesn't really have analytical meaning. Sometimes, however, we rename it to school_year in stg, when school_year isn't available in the source ed-fi endpoint.

maybe that's something we should add to the stg model for candidates.. like we do in stuEdOrg

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this comes down to a decision: do we want to treat candidates like students, in that there's one record per candidate + year, or like staffs, in that there's one record per candidate. Maybe this is worth asking at dev meeting

Defines an educator candidate.

##### Primary Key:
`k_candidate` - There is one record per candidate
Copy link
Collaborator

Choose a reason for hiding this comment

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

technically candidate and year, right? assuming ODSes are single-year

Copy link
Collaborator

Choose a reason for hiding this comment

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

we should probably apply some of the same transformations that we apply to staff & students, to candidates. for example:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants