Skip to content

feature/:Enhanced dim_course_section to expose section_characteristics descriptors#231

Open
smckee-tnedu wants to merge 1 commit into
edanalytics:mainfrom
tnedu:feature/section_characteristics_in_dim_course_section
Open

feature/:Enhanced dim_course_section to expose section_characteristics descriptors#231
smckee-tnedu wants to merge 1 commit into
edanalytics:mainfrom
tnedu:feature/section_characteristics_in_dim_course_section

Conversation

@smckee-tnedu
Copy link
Copy Markdown
Contributor

Description & motivation

After the fix for course_level_characteristics, the visibility of section characteristics was gone. This set of changes brings that visibility back.

Breaking changes introduced by this PR:

This change requires a new xwalk: xwalk_section_characteristics.csv. An example looks like this:

section_characteristic_descriptor,indicator_name
Fall Block,is_fall_block
Spring Block,is_spring_block
Year-long,is_year_long

PR Merge Priority:

  • Low
  • Medium
  • High

Changes to existing files:

  • models/core_warehouse/dim_course_section.sql : Uses a new build model to expose the descriptors that come in on the sections_characteristics field.

New files created:

  • models/build/edfi_3/courses/bld_ef3__section__wide_section_characteristics.sql : Builds a set of wide columns that can be accordioned in dim_course_section.

Tests and QC done:

This has been tested within stadium_tennessee.

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

@rlittle08 rlittle08 requested a review from AngelicaLastra May 8, 2026 18:01
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.

1 participant