-
-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UBI Implementation #1235
UBI Implementation #1235
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1235 +/- ##
==========================================
- Coverage 99.2% 99.03% -0.18%
==========================================
Files 38 38
Lines 2650 2689 +39
==========================================
+ Hits 2629 2663 +34
- Misses 21 26 +5
Continue to review full report at Codecov.
|
@andersonfrailey said about the UBI benefits PR #1235:
Well, I guess the proper name for this parameter all depends on what you view as the "normal" situation. Searching through
This is not what If you look at the Contributor Guide's parameter naming rules, it seems to me that what we are dealing with here is an AGI exclusion rate, the fraction of UBI benefits that are excluded from AGI. Those rules suggest the new parameter might better be named Using the
and the long_name and description would be:
Does anybody have a better idea? @MattHJensen @feenberg @Amy-Xu @GoFroggyRun @codykallen @zrisher |
@martinholmer, I like your suggestion more than using |
@andersonfrailey, pull request #1235 looks very good now except for maybe one thing. I'm not familiar with the UBI conversation, but wouldn't most people expect that UBI benefits would be included in AGI? Right now, you have in Once we decide on the best default value, I'll be happy to merge #1235 into the master branch. |
@martinholmer, good catch. I haven't seen any true consensus among UBI proponents on whether or not it should be taxed, but I agree with your logic that most wouldn't be in favor of handing Bill Gates tax-free money. I'll make the change so it's ready to merge. |
@andersonfrailey, |
Second time's the charm. This PR adds a feature to Tax-Calc that calculates the effects of a universal basic income. The UBI function relies on three new PUF variables (
nu18
,n1821
, andn21
) that hold the number of people in each age range in each tax unit. These have been discussed in TaxData PR #83.Four new parameters are added into
current_law_policy.json
._UBI1-3
allow the user to specify different levels of UBI for each age group._UBI_hc
allows the user to specify what portion of the UBI is taxable (I'm not surehc
is the correct tag for this, but it seemed like the best option. If this needs to change let me know).Using the new variables and parameters, the function calculates three values:
ubi
: total income from the UBI received by the tax unit.taxable_ubi
: the portion of total UBI that is taxable. This is added to AGI.nontaxable_ubi
: the portion of total UBI that is not taxable. This is added to expanded income.Adding only taxable UBI to AGI ensures that if the user specifies that UBI is not taxable, tax liability for the unit does not change. Nontaxable UBI, rather than total UBI is added to expanded income so that the taxable portion of UBI, which was already added to AGI, is not double counted.
@MattHJensen @Amy-Xu @martinholmer