Skip to content

fix: Charlson age_score half-open decade bands - #2129

Merged
alistairewj merged 1 commit into
MIT-LCP:mainfrom
Chessing234:fix/charlson-age-score-decade-boundary
Aug 9, 2026
Merged

fix: Charlson age_score half-open decade bands#2129
alistairewj merged 1 commit into
MIT-LCP:mainfrom
Chessing234:fix/charlson-age-score-decade-boundary

Conversation

@Chessing234

@Chessing234 Chessing234 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Charlson age_score decade bands are half-open so boundary ages are not double-counted

Split from the too-large #2124 per maintainer feedback.

Test plan

  • sqlfluff on charlson.sql dialects

@Chessing234
Chessing234 force-pushed the fix/charlson-age-score-decade-boundary branch from 3d56027 to 695a7ab Compare August 5, 2026 10:34
@alistairewj

Copy link
Copy Markdown
Member

please rebase the sqlfluff changes

@alistairewj

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request, rebasing should remove all changes to the yml

age<=50 scored 0; CCI awards 1 point starting at age 50.

Signed-off-by: Taksh <takshkothari09@gmail.com>
@Chessing234
Chessing234 force-pushed the fix/charlson-age-score-decade-boundary branch from 695a7ab to 25f6c47 Compare August 9, 2026 13:09
@Chessing234

Copy link
Copy Markdown
Contributor Author

rebased — the yml hunks are gone, main already has the fork-annotate fix from #2148. down to the age_score bands in the three charlson dialects.


Generated by Claude Code

@alistairewj
alistairewj merged commit 20167f7 into MIT-LCP:main Aug 9, 2026
12 checks passed
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.

2 participants