Skip to content

Refactor income imputation and remove winter fuel allowance from loss calculations#211

Merged
nikhilwoodruff merged 1 commit into
mainfrom
refactor-income-imputation
Oct 21, 2025
Merged

Refactor income imputation and remove winter fuel allowance from loss calculations#211
nikhilwoodruff merged 1 commit into
mainfrom
refactor-income-imputation

Conversation

@nikhilwoodruff

Copy link
Copy Markdown
Contributor

Fixes #210

This refactors the income imputation logic to be more selective about what gets imputed on the main dataset.

Changes:

  • Extracted imputation logic into a reusable impute_over_incomes function
  • Now only imputes dividend income on the main dataset (while still imputing all income components on the zero-weight copy)
  • Removed winter fuel allowance from loss calculations (both count and total metrics)

This should improve accuracy by being more targeted about which income components need imputation on the full dataset.

@nikhilwoodruff nikhilwoodruff merged commit 048f389 into main Oct 21, 2025
3 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.

Refactor income imputation and remove winter fuel allowance from loss calculations

1 participant