Skip to content

Feature/consecutive absences count #196

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 23 commits into
base: main
Choose a base branch
from
Draft

Feature/consecutive absences count #196

wants to merge 23 commits into from

Conversation

@gnguyen87
Copy link
Contributor

@gnguyen87 gnguyen87 commented Nov 6, 2025 •
edited
Loading

Description & motivation

Calculate a running record of the n-th consecutive unexcused absence per student.

Breaking changes introduced by this PR:

PR Merge Priority:

  • Low
  • Medium
  • High

Changes to existing files:

fct_student_attendance

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

gnguyen87 and others added 17 commits January 24, 2025 11:32
Bump/0.4.0 (#154)
d092a6b 
* Update dbt_project.yml

* Update packages.yml

* Update CHANGELOG.md
Add fct model that has a consecutive day count column for student att…
0d823bb 
…endance/absences
Add code doc
e38f7d3
Update CHANGELOG
9dcd9b0
@gnguyen87
Update fct_student_consecutive_attendance_count.yml
60e528e
import consecutive attendance count into existing fct_student_daily_a…
edd944a 
…ttendance model
clean up code + add docs
569b6a5
refactor model
d5c3b51
remove unnecessary join and add docs
82008b9
filter to only account for unexcused absences
2b2e957
retire outdated model
f50e995
add docs
27469e7
Merge remote-tracking branch 'origin' into feature/consecutive_absenc…
fa01a92 
…es_count
distinction between unexcused vs other att event
1bceff6
distinction between all attendance events
2034eae
update: distinction between unexcused absence vs att events
a1a07a6
update: distinction between all att events
ce4bb2e
@gnguyen87 gnguyen87 force-pushed the feature/consecutive_absences_count branch from 6323440 to ce4bb2e Compare November 7, 2025 20:43
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.

2 participants

@gnguyen87