-
Notifications
You must be signed in to change notification settings - Fork 201
DC Medicaid program for immigrants over 21 #6541
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
base: main
Are you sure you want to change the base?
DC Medicaid program for immigrants over 21 #6541
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6541 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 9 -2
Lines 146 151 +5
=========================================
+ Hits 146 151 +5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…etails for undocumented immigrants.
policyengine_us/parameters/gov/states/dc/dhcf/medicaid/eligibility/adult_income_limit.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/parameters/gov/states/dc/dhcf/medicaid/eligibility/adult_income_limit.yaml
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/dc/dhcf/medicaid/eligibility/dc_medicaid_age_eligible.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/dc/dhcf/medicaid/income/dc_medicaid_income_eligible.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/dc/dhcf/medicaid/income/dc_medicaid_income_eligible.py
Show resolved
Hide resolved
- Introduced `dc_medicaid_cost` and `dc_medicaid_cost_if_enrolled` variables. - Updated `healthcare_benefit_value` to include `dc_medicaid_cost`. - Removed deprecated immigration eligibility tests and refined logic for DC Medicaid eligibility. - Enhanced documentation for clarity on DC Health Care Alliance coverage.
This PR should only contain Colorado OmniSalud implementation. DC Medicaid/Alliance code is already in PR PolicyEngine#6541. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
PR Review: DC Medicaid for Immigrants🔴 Critical (Must Fix)
🟡 Should Address
🟢 Suggestions
Validation Summary
What's Good
Next StepsTo auto-fix critical issues: Or address manually:
🤖 Generated with Claude Code |
- Replace hard-coded age < 19 with child_max_age parameter in cost file - Fix array mutation bug by using where() instead of in-place assignment - Update broken reference URLs (dhcf.dc.gov/alliance → dhcf.dc.gov/node/1796971) - Add KFF references to corroborate 324% FPL values for children/pregnant - Rename test files to match convention (*_test.yaml → *.yaml) - Add 14 boundary tests for income limits and age transitions - Update cost tests to reflect DC's child_max_age of 20 (not 19) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fixes AppliedAll issues from the review have been addressed: 🔴 Critical Issues Fixed
🟡 Should Address Fixed
Test ResultsAll 66 tests pass for DC Medicaid. Files Modified
🤖 Generated with Claude Code |
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Resolved conflicts in: - policyengine_us/variables/gov/hhs/medicaid/eligibility/is_medicaid_eligible.py - uv.lock (regenerated) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Upgraded uv to match CI version, fixes revision mismatch. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
| @@ -0,0 +1,16 @@ | |||
| description: DC limits Medicaid/Alliance for adults (21+) to this percentage of the Federal Poverty Level | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| description: DC limits Medicaid/Alliance for adults (21+) to this percentage of the Federal Poverty Level | |
| description: The District of Columbia limits Medicaid/Alliance for adults (21+) to this percentage of the Federal Poverty Level. |
And other description in parameter files.
| 2021-01-01: false | ||
| DC: | ||
| 2021-01-01: true | ||
| 2021-01-01: false # DC now handles undocumented coverage through dc_medicaid_eligible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a new rule starting in 2021-01? I am thinking may be we should put it as
20xx-01-01: true
2021-01-01: false
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2021 is just when we start the parameters from, this false means that we use state specific logic. the real start dates are in the dc folder.
...yengine_us/variables/gov/hhs/medicaid/eligibility/is_medicaid_immigration_status_eligible.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/dc/dhcf/medicaid/costs/dc_medicaid_cost_if_enrolled.py
Outdated
Show resolved
Hide resolved
policyengine_us/variables/gov/states/dc/dhcf/medicaid/income/dc_medicaid_income_eligible.py
Outdated
Show resolved
Hide resolved
| ) | ||
|
|
||
|
|
||
| class dc_medicaid_cost_if_enrolled(Variable): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This variable is too long, consider file an issue to refactor the medicaid_cost_if_enrolled, then we can reuse the same logic without so many duplicates code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fixes #6365