Repository navigation
Turn off ACA take-up for TAXSIM records - #1518
Merged
Merged
Conversation
TAXSIM has no health coverage input. PE-US assigns a premium tax credit to every eligible tax unit by default, and since PE-US #10093 Oregon's federal tax liability subtraction nets it out, which moved 53 census records and broke test_or_kicker_output on main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
TAXSIM has no health-coverage input, so a TAXSIM record carries no premium tax credit. PE-US defaults
takes_up_aca_if_eligibleto True and assigns a premium tax credit to every eligible tax unit. Since PolicyEngine/policyengine-us#10093, Oregon's federal tax liability subtraction nets outassigned_aca_ptc, so the imputed credit now changessiitax.This PR sets
takes_up_aca_if_eligibleto False for every TAXSIM record, in both paths:core/input_mapper.py(single household)runners/policyengine_runner.py(Microsimulation / CLI)Effect
test_or_kicker_output[2025]fails onmaintoday: the imputed credit is $9,729.79, so the OR subtraction falls from 8,500 to 5,673.85. With this PR the test passes again.siitaxas PolicyEngine before #10093.assigned_aca_ptcreaches tax variables only throughor_federal_tax_liability_subtractionandincome_tax_excluding_ptc(which the emulator does not output). Alabama's federal tax deduction readsnet_premium_tax_credit, an input with default 0 that take-up does not change.Tests
tests/test_aca_takeup.pycovers both paths (OR 2025 joint couple).tests/test_mappers.pyandtests/test_mtr_branch_overrides.pyupdated for the new input.This unblocks CI on #1516 and #1517.
🤖 Generated with Claude Code