PolicyEngine / PolicyEngine/policyengine-us

Refactor the `regular_tax_before_credits` variable and add unit tests

Open
#4,817 2 comments 0 reactions 1 assignee Assigned to @PavelMakarchuk View on GitHub
AMT
Dominant language
Python
Stars
163
Forks
213
Avg merge
3d 3h
Merged PRs (30d)
96

Description

It currently includes a number of variables which are not descriptive as well as calculation steps which do not seem transparent

First step would be to compare the calculation to `capital_gains_tax` (up until the inclusion of `income_tax_main_rates`) and to identify which steps can be consolidated for clarity

Secondly, unit tests should be added to test the file - it will be not done in #4815

We should also add references and tie the hard coded `0.28` parameter from the `dwks41` calculation to the amt bracket parameter which is refactored in #4815

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.