Skip to content

Stop income imputation scaling housing costs#375

Draft
MaxGhenis wants to merge 4 commits intomainfrom
codex/remove-income-housing-scaling
Draft

Stop income imputation scaling housing costs#375
MaxGhenis wants to merge 4 commits intomainfrom
codex/remove-income-housing-scaling

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • remove the SPI income-imputation adjustment that scaled rent and mortgage costs by total imputed income growth
  • keep Gift Aid outputs separate from income components without tying them to housing-cost rescaling
  • replace the rescale-factor helper tests with a behavior test that asserts rent and mortgage costs are preserved while income outputs are overwritten

Rationale

Housing costs should come from the housing data path, calibration, and uprating logic. SPI income imputation changes person income components; it should not mechanically multiply household rent or mortgage costs as a side effect.

Tests

  • uv run pytest policyengine_uk_data/tests/test_income_imputation_housing_costs.py policyengine_uk_data/tests/test_frs_only_imputation.py policyengine_uk_data/tests/test_housing_targets.py -q
  • uv run ruff check policyengine_uk_data/datasets/imputations/income.py policyengine_uk_data/tests/test_income_imputation_housing_costs.py

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.

1 participant