PolicyEngine / PolicyEngine/policyengine-us

Hide or relabel the `spm_unit_taxes` variable

Open
#3,450 0 comments 0 reactions 0 assignees View on GitHub
tax
Dominant language
Python
Stars
162
Forks
212
Avg merge
3d 3h
Merged PRs (30d)
96

Description

This is what shows up when searching for `Taxes`

image

This deviates from the right side because it comes from [spm_unit_taxes](https://github.com/PolicyEngine/policyengine-us/blob/master/policyengine_us/variables/household/income/spm_unit/spm_unit_taxes.py), which itself comes from [spm_unit_state_tax](https://github.com/PolicyEngine/policyengine-us/blob/master/policyengine_us/variables/household/expense/tax/spm_unit_state_tax.py), which comes from state_income_tax, which omits many states to avoid circular references:

https://github.com/PolicyEngine/policyengine-us/blob/502c2e9f059c071af2a2deacf0aca529d8f5a6eb/policyengine_us/variables/gov/states/tax/income/state_income_tax.py#L16

The quickest thing we can do here is to create a new `state_income_taxes_paid` variable, which is used for the SALT deduction and can have the omitted states as written here. Then `state_income_tax` can be used as an output variable with all states included. But in the meantime we could relabel or hide this variable which is giving incorrect information relative to how we display it.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.