Skip to content

Map CPS farm income to farm operations#827

Closed
MaxGhenis wants to merge 1 commit intoPolicyEngine:mainfrom
MaxGhenis:codex/fix-cps-farm-operations-income
Closed

Map CPS farm income to farm operations#827
MaxGhenis wants to merge 1 commit intoPolicyEngine:mainfrom
MaxGhenis:codex/fix-cps-farm-operations-income

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • Map CPS FRSE_VAL farm self-employment income to farm_operations_income instead of farm_income.
  • Add a focused regression test for CPS personal income construction.
  • Add a towncrier changelog fragment.

Root cause

PolicyEngine distinguishes Schedule J-style farm_income from Schedule F farm operations income. The PUF pipeline already maps these separately, but the raw CPS builder was storing FRSE_VAL in farm_income, which makes CPS inconsistent with the model semantics and the PolicyEngine US SECA fix.

Tests

  • uv run python -m pytest tests/unit/datasets/test_cps_income_variables.py -q
  • uv run ruff check policyengine_us_data/datasets/cps/cps.py tests/unit/datasets/test_cps_income_variables.py
  • uv run python -m pytest tests/unit/datasets/test_cps_income_variables.py tests/unit/datasets/test_cps_tax_unit_construction.py tests/unit/datasets/test_cps_file_handles.py -q

@MaxGhenis
Copy link
Copy Markdown
Contributor Author

Replaced by #828 from a PolicyEngine org branch so CI can run.

@MaxGhenis MaxGhenis closed this Apr 25, 2026
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