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

Threshold for self-employment income exempt from SECA taxes #2659

Merged
merged 9 commits into from
Jun 24, 2022

Conversation

jdebacker
Copy link
Member

@jdebacker jdebacker commented May 9, 2022

Adds parameters for threshold on self-employment income subject to SECA taxes, addressing Issue #2656.

@jdebacker
Copy link
Member Author

jdebacker commented May 26, 2022

The changes here results in some differences in the comparison files that show increased differences between T-C and SOI tabulations when using CPS data (also some small diffs with the PUF). For example, the "expected" file before this PR shows:

TABLE for ITAX
AGI category           	     T-C	     SOI	 %diff
[-9e+99, 1)            	  -0.020	   0.242	-108.0
[1, 5000)              	  -1.630	   0.041	-4084.5
[5000, 10000)          	  -7.181	   0.368	-2051.4

After the changes proposed here, this becomes:

TABLE for ITAX
AGI category           	     T-C	     SOI	 %diff
[-9e+99, 1)            	  -0.020	   0.242	-108.0
[1, 5000)              	  -1.630	   0.041	-4084.7
[5000, 10000)          	  -7.181	   0.368	-2051.4

I'm not sure if this is something to be concerned about.

@codecov
Copy link

codecov bot commented May 26, 2022

Codecov Report

Merging #2659 (8969262) into master (9774b47) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2659   +/-   ##
=======================================
  Coverage   98.54%   98.54%           
=======================================
  Files          14       14           
  Lines        2609     2609           
=======================================
  Hits         2571     2571           
  Misses         38       38           
Flag Coverage Δ
unittests 98.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@jdebacker
Copy link
Member Author

@MaxGhenis Does this PR look like a good response to Issue #2656 to you?

@MaxGhenis
Copy link
Contributor

LGTM thanks @jdebacker!

@jdebacker
Copy link
Member Author

@MattHJensen can you review this PR and merge if it looks good to you? Thanks!

@MattHJensen
Copy link
Contributor

Thanks for reviewing with me on the TC meeting today. Merging.

@MattHJensen MattHJensen merged commit 98b23dc into PSLmodels:master Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants