Skip to content
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

Does nu18 or n1821 include 18-year-olds? #1669

Closed
MaxGhenis opened this issue Nov 16, 2017 · 4 comments
Closed

Does nu18 or n1821 include 18-year-olds? #1669

MaxGhenis opened this issue Nov 16, 2017 · 4 comments
Assignees
Labels

Comments

@MaxGhenis
Copy link
Contributor

MaxGhenis commented Nov 16, 2017

The CPS records_variables.json file describes nu18 as

Number of people under 18 years old in the filing unit.

, n1821 as

Number of people over 18 and under 21 years old in the filing unit

and n21 as

Number of people 21 years old or older in the filing unit

Based on this language I'm not clear on whether nu18 or n1821 includes 18-year-olds.

@MaxGhenis
Copy link
Contributor Author

The UBI parameters describe _UBI1 as

UBI benefit for those under 18

and _UBI2 as

UBI benefit for those 18 through 20

@martinholmer
Copy link
Collaborator

martinholmer commented Nov 16, 2017

@MaxGhenis, Thanks for the documentation bug report.
Pull request #1672 clarifies that n1821 is number of people age 18-20.

@martinholmer
Copy link
Collaborator

@MaxGhenis, In the next version of Tax-Calculator we'll change the variable n1821 to be n1820 so that the variable name says what it means. Again, thanks for your bug report.

@MattHJensen @andersonfrailey

@martinholmer
Copy link
Collaborator

The merge of pull request #1871 completes the renaming of the old data variable n1821 to n8120.
Now the more accurate name is used both in the latest PUF data file and in the latest CPS data file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants