-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/discipline updates #75
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description & motivation
After seeing multiple examples of how SIS' map their discipline data to the flexible discipline domain and discussing reporting use-cases with the podium team and other implementations, we realized that we needed to update the discipline data model in the warehouse.
Due to the many-to-many relationship between discipline action events/associated disciplines and discipline incidents/associated behaviors and the oftentimes not-analytically meaningful association to IDs, we wanted the ability to report all disciplines/behaviors for a single discipline action event and all disciplines/behaviors for a single discipline incident in new, separated models. These will replace the single model that includes both
k_discipline_incident
andk_discipline_actions_event
.In addition, we did not previously give users the ability to join all discipline action events/disciplines to all discipline incidents/behaviors (which would represent a different use-case than what is offered via the summary tables), so we are adding a key to join
fct_student_discipline_actions
andfct_student_discipline_incident_behaviors
.PR Merge Priority:
Changes to existing files:
dim_discipline_incidents
: No changes to the structure, but name is updated todim_discipline_incident
to match the singular pattern of other dimensions. All models that reference this model were also updated to match the new naming convention.fct_student_discipline_actions
: Carriedschool_year
into the model, added an array ofk_student_discipline_incident_behavior
to allow for a join againstfct_student_discipline_incident_behaviors
. Also added example queries to the model documentation.fct_student_discipline_incident_behaviors
: Added a new keyk_student_discipline_incident_behavior
for joins againstfct_student_discipline_actions
.fct_student_discipline_incident_behaviors_actions
: deleted.New files created:
bld_ef3__stu_discipline_action__stu_incident_behavior
: The purpose of this model is to create an array of keys of student discipline incident behaviors that are linked to a single discipline action event, for use infct_student_discipline_actions
.fct_student_discipline_incident_summary
: The purpose of this model is to create a table at the grain of a student and discipline incident that includes an array of all behaviors associated to that incident and an array of all discipline actions associated to that incident regardless of whether or not those discipline actions are associated to multiple action events.fct_student_discipline_actions_summary
: The purpose of this model is to create a table at the grain of a student and discipline actions event that includes an array of all discipline actions associated to that actions event and an array of all behaviors associated to that event regardless of whether or not those behaviors are associated to multiple incidents.Tests and QC done:
fct_student_discipline_actions_summary
and pointed a development version of Podium to the new model. They then QC'd the output and gave a thumbs up that everything looked as expected.NOTE: This branch includes breaking changes