Closed martinholmer closed 8 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
552b2f8
) 98.54% compared to head (fe2ff9a
) 98.54%. Report is 1 commits behind head on master.
Thanks for this PR @martinholmer! Merging.
This pull request contains a few changes that will make it easier to conduct validation tests of new datasets being developed for use by Tax-Calculator. These changes have no effect on the taxes calculated by the model.
In the
Makefile
change the logic of theclean
target to correspond to the existing logic in thepackage
target.In the
Makefile
add the--disable-warnings
option to just thepytest-cps
target.Add a
SS_all_in_agi
boolean parameter to thepolicy_current_law.json
file and use it in theSSBenefits
function in thecalcfunctions.py
module. This parameter permits the estimation of the tax expenditure associated with the less-than-complete taxation of social security benefits.Promote a local variable in the
GainsTax
function in thecalcfunctions.py
module to be a global variable defined in therecords_variables.json
file. This variable makes it easier to check that the existingCG_nodiff
parameter is doing what it is designed to do.Suppress a pandas deprecation warning in the
taxcalcio.py
module.Break up several overly long lines in the
pytest_sessionfinish
function in theconftest.py
module.