Skip to content

Commit 3d3b125

Browse files
hua7450claude
andcommitted
Review-fix round 6: clear final should-address items
- Spell out "Employment-Sponsored Subsidy" instead of "ESS" acronym in 3 parameter descriptions (ess_window_months, activity_hours_ess_year_1, activity_hours_ess_year_2). - Simplify hhs_smi access in is_ar_sra_income_eligible and ar_sra_income_tier: drop the explicit period.this_year + /MONTHS_IN_YEAR pattern in favor of reading the YEAR-defined hhs_smi at MONTH period (core auto-converts the USD flow variable annual->monthly). 146 AR SRA tests pass; all critical and should-address findings from multi-round /review-program now cleared. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 157bdad commit 3d3b125

5 files changed

Lines changed: 5 additions & 7 deletions

File tree

policyengine_us/parameters/gov/states/ar/ade/oec/sra/eligibility/activity_hours_ess_year_1.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
description: Arkansas sets this minimum weekly activity hours threshold for at least one adult in an ESS Year 1 household under the School Readiness Assistance program.
1+
description: Arkansas sets this minimum weekly activity hours threshold for at least one adult in an Employment-Sponsored Subsidy Year 1 household under the School Readiness Assistance program.
22
values:
33
2025-10-01: 20
44

policyengine_us/parameters/gov/states/ar/ade/oec/sra/eligibility/activity_hours_ess_year_2.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
description: Arkansas sets this minimum weekly activity hours threshold for at least one adult in an ESS Year 2 household under the School Readiness Assistance program.
1+
description: Arkansas sets this minimum weekly activity hours threshold for at least one adult in an Employment-Sponsored Subsidy Year 2 household under the School Readiness Assistance program.
22
values:
33
2025-10-01: 25
44

policyengine_us/parameters/gov/states/ar/ade/oec/sra/eligibility/ess_window_months.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
description: Arkansas limits the ESS post-Transitional Employment Assistance window to this many months under the School Readiness Assistance program.
1+
description: Arkansas limits the Employment-Sponsored Subsidy post-Transitional Employment Assistance window to this many months under the School Readiness Assistance program.
22
values:
33
2025-10-01: 24
44

policyengine_us/variables/gov/states/ar/ade/oec/sra/ar_sra_income_tier.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class ar_sra_income_tier(Variable):
2020
def formula(spm_unit, period, parameters):
2121
p = parameters(period).gov.states.ar.ade.oec.sra
2222
monthly_income = spm_unit("ar_sra_countable_income", period)
23-
monthly_smi = spm_unit("hhs_smi", period.this_year) / MONTHS_IN_YEAR
23+
monthly_smi = spm_unit("hhs_smi", period)
2424
ratio = where(monthly_smi > 0, monthly_income / monthly_smi, np.inf)
2525
return select(
2626
[

policyengine_us/variables/gov/states/ar/ade/oec/sra/eligibility/is_ar_sra_income_eligible.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,5 @@ class is_ar_sra_income_eligible(Variable):
1212
def formula(spm_unit, period, parameters):
1313
p = parameters(period).gov.states.ar.ade.oec.sra.eligibility
1414
monthly_income = spm_unit("ar_sra_countable_income", period)
15-
monthly_smi_limit = (
16-
spm_unit("hhs_smi", period.this_year) * p.income_smi_rate / MONTHS_IN_YEAR
17-
)
15+
monthly_smi_limit = spm_unit("hhs_smi", period) * p.income_smi_rate
1816
return monthly_income <= monthly_smi_limit

0 commit comments

Comments
 (0)