Skip to content

Add course characteristics to dim_course#245

Open
ryanaguilar wants to merge 3 commits into
mainfrom
feature/course_chars_course
Open

Add course characteristics to dim_course#245
ryanaguilar wants to merge 3 commits into
mainfrom
feature/course_chars_course

Conversation

@ryanaguilar

@ryanaguilar ryanaguilar commented May 18, 2026

Copy link
Copy Markdown
Contributor

Description & motivation

This PR would add a course charactersitics array to the dim_course model. This array already exists in the course section model, but would now be surfaced at the course level as well. Addresses this Boston Monday item.

Breaking changes introduced by this PR:

No breaking changes.

PR Merge Priority:

  • Low
  • Medium
  • High

Changes to existing files:

  • dim_course : Add course level characteristics array

New files created:

Tests and QC done:

  • Tested in stadium_boston

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 ryanaguilar requested a review from rlittle08 June 5, 2026 12:50
@ryanaguilar ryanaguilar closed this Jun 5, 2026
@ryanaguilar ryanaguilar reopened this Jun 5, 2026
@ryanaguilar ryanaguilar marked this pull request as ready for review June 5, 2026 12:50
@rlittle08

Copy link
Copy Markdown
Collaborator

marking for 0.7.0 so it can be aligned with release of #231. Very likely to be released ~August

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.

2 participants