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

Change default start_year to 2018 #749

Open
hdoupe opened this issue Nov 21, 2017 · 6 comments
Open

Change default start_year to 2018 #749

hdoupe opened this issue Nov 21, 2017 · 6 comments

Comments

@hdoupe
Copy link
Collaborator

hdoupe commented Nov 21, 2017

Should the default start_year be changed to 2018?

cc @martinholmer @MattHJensen

@MattHJensen
Copy link
Contributor

I would recommend leaving it at 2017 until tax-calculator updates to the IRS-published 2018 parameter values that conform to the IRS rounding rules.

@martinholmer
Copy link
Contributor

@hdoupe, Will the TCJA reforms work in the TaxBrain GUI with a start year of 2018?
Don't all those reforms have a provision (the cpi_offset parameter) that is implemented in 2017?

@MattHJensen
Copy link
Contributor

MattHJensen commented Mar 8, 2018

@martinholmer asked:

Will the TCJA reforms work in the TaxBrain GUI with a start year of 2018?
Don't all those reforms have a provision (the cpi_offset parameter) that is implemented in 2017?

I believe that this feature will solve this issue:

You may specify a parameter change in the year before the Start Year with the < operator. For example, if you set the Start Year as 2018 and a Parameter Indexing Offset parameter to <,-0.0025,-0.001 then this sets the Offset to -0.0025 in 2017 and -0.001 in 2018.


I would recommend leaving it at 2017 until tax-calculator updates to the IRS-published 2018 parameter values that conform to the IRS rounding rules.

This is still my recommendation. The relevant Tax-Calculator issue is PSLmodels/Tax-Calculator#1694 (comment)

@hdoupe
Copy link
Collaborator Author

hdoupe commented Mar 8, 2018

@MattHJensen thanks for the update on the start year. Please let me know when you are ready to change it to 2018.

@hdoupe
Copy link
Collaborator Author

hdoupe commented Mar 8, 2018

Also, the < operator resolves the issue with the cpi_offset parameter. See #790

@MaxGhenis
Copy link
Contributor

Can this happen? I think users are no longer expecting to compare against pre-TCJA law...

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

No branches or pull requests

4 participants