-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
[WIP] itemized deduction surtax and itemized deduction haircut consistency #490
[WIP] itemized deduction surtax and itemized deduction haircut consistency #490
Conversation
I just fixed one issue, which was that This did not entirely resolve our inconsistency between the haircuts and the surtax. |
After 982e350 there is now only one record that is inconsistent. In the puf-cps merged file, it is row 125,850 and recid 201,009. |
It seems that recid 201,009 is not optimizing their deductions properly. With itemized deductions allowed, the household's combined taxes are 121700.46. Without itemized deductions allowed, their combined tax falls to 121300.94. cc @GoFroggyRun. |
On Mon, 30 Nov 2015, Matt Jensen wrote:
Without access to the record, I can't be sure what is happening, but this
|
After investigating the unit in the puf file with cc @MattHJensen |
If you would send me the data and calculated values I can analyze why this dan On Wed, 2 Dec 2015, Sean.Wang wrote:
|
@feenberg , thanks for your advice. The thing about current python calculator is that, we are using the costly way to decide whether a unit should itemize or not. More specifically, we compared the income tax before credit amount, |
On Wed, 2 Dec 2015, Sean.Wang wrote:
Unless it is an issue with credits, then I think there is a problem that dan
|
Closing this in favor of #749. cc @GoFroggyRun and @martinholmer. |
This PR adds a test to ensure that corresponding itemized deduction haircuts and an itemized deduction surtax produce consistent results per #488.
It also attempts to fix the known inconsistency described in #487 in order to get the test to pass by adding the surtax to _combined as well as to _iitax.
This is a major improvement over current master, but it is still not perfect. Currently _combined under the two scenarios are not almost_equal to two decimals for 0.0732779677577% of records. I am still exploring why these don't match.