Closed MaxGhenis closed 3 years ago
The previously described issue was identified as assigning new (flat) tax liabilities to persons based on their personal AGI rather than the SPM unit's AGI, i.e. these lines:
target_persons['new_tax'] = target_persons.agi * flat_tax
# ...
target_persons['new_spm_resources'] = (target_persons.spm_resources +
target_persons.hh_current_tax +
target_persons.hh_fica -
target_persons.new_tax + # HERE
target_persons.hh_ubi)
This explains the new results looking less favorable to the UBI scenarios (tax was underestimated).
@ngpsu22 wdyt of using tax after credit rather than before? This could be simpler to explain (I'm also still not entirely clear if EITC and ACTC are the only differences; IPUMS documentation on FEDTAX
and FEDTAXAC
and forum responses don't leave me certain).
Actually I think it has to be after credits, since credits aren't only EITC and ACTC, they're also nonrefundable credits which can't be separated from the rest of the tax code. This means slightly higher rates are required to achieve poverty- and inequality-neutrality. Updated accordingly in the latest commit.
Some to-dos before publishing:
Couple other things to mention:
@ngpsu22 think this is just about ready, only thing left is making the hover labels look nice on the new charts and adding some detail on the OECD data. LMK if you have other feedback before I post it.
By @ngpsu22
Three remaining issues: