Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[US_MI][Workflows] Take the greatest of sentencing length for early d…
…ischarge from parole criteria (Recidiviz/recidiviz-data#29250) ## Description of the change We got feedback from trusted testers that clients were ineligible based on having a current offense with `(statute LIKE "750.110" OR statute LIKE "750.110A%")` and a sentence length > 15 years. Here we were calculating the date based on the difference between the `DATE_DIFF(projected_completion_date_max,effective_date,DAY)` which was giving us just under 15 years, despite the `max_sentence_length_days_calculated` being over 15 years. To resolve this issue we take the `GREATEST` of these fields. Tested in `shuff`. This change results in 14 fewer clients eligible for early discharge from parole out of 1187 clients. ## Type of change > All pull requests must have at least one of the following labels applied (otherwise the PR will fail): | Label | Description | |----------------------------- |----------------------------------------------------------------------------------------------------------- | | Type: Bug | non-breaking change that fixes an issue | | Type: Feature | non-breaking change that adds functionality | | Type: Breaking Change | fix or feature that would cause existing functionality to not work as expected | | Type: Non-breaking refactor | change addresses some tech debt item or prepares for a later change, but does not change functionality | | Type: Configuration Change | adjusts configuration to achieve some end related to functionality, development, performance, or security | | Type: Dependency Upgrade | upgrades a project dependency - these changes are not included in release notes | ## Related issues Closes #Recidiviz/recidiviz-dashboards#5201 ## Checklists ### Development **This box MUST be checked by the submitter prior to merging**: - [x] **Double- and triple-checked that there is no Personally Identifiable Information (PII) being mistakenly added in this pull request** These boxes should be checked by the submitter prior to merging: - [ ] Tests have been written to cover the code changed/added as part of this pull request ### Code review These boxes should be checked by reviewers prior to merging: - [ ] This pull request has a descriptive title and information useful to a reviewer - [ ] Potential security implications or infrastructural changes have been considered, if relevant GitOrigin-RevId: e4428f8fad8e6debc64e259faeba4e068c5a3e81
- Loading branch information