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

Itemized deduction ceiling switch not correctly processed by TaxBrain GUI interface #598

Closed
hdoupe opened this issue Aug 2, 2017 · 5 comments
Labels
Milestone

Comments

@hdoupe
Copy link
Collaborator

hdoupe commented Aug 2, 2017

I noticed that setting the switches in Deductions subject to the cap on itemized deduction benefits to false did not actually cause the parameter, _ID_BenefitCap_Switch, that was passed to taxcalc to be false. I ran this reform on a local instance of taxbrain and the value of _ID_BenefitCap_Switch was passed to taxcalc as

{... u'_ID_BenefitCap_Switch': [[True, True, False, True, 1.0, 1.0, 1.0]], ...}

However, it should have been
{... u'_ID_BenefitCap_Switch': [[1.0, 1.0, 0.0, 1.0, 0.0, 1.0, 1.0]], ...} or
{... u'_ID_BenefitCap_Switch': [[True, True, False, True, False, True, True]], ...}

But when I upload the reform as a file, ID_BenefitCap_Switch is passed back as:

{...u'_ID_BenefitCap_Switch': [[1.0, 1.0, 0.0, 1.0, 0.0, 1.0, 1.0]], ...}

which is what I expected.

Here is a link to the test app run of the reform file: http://ospc-taxes7.herokuapp.com/taxbrain/1389/

And here are the contents of the reform file:

{
    "policy": {
        "_ID_BenefitCap_Switch":
            {"2017": [[1.0, 1.0, 0.0, 1.0, 0.0, 1.0, 1.0]]},
        "_ID_c":
            {"2017": [[20000, 40000, 20000, 20000, 99e9]]}
    }
}

So, it appears that there is an issue with the GUI interface to Taxbrain, but the file upload functionality appears to work correctly.

@brittainhard @PeterDSteinberg

@hdoupe hdoupe changed the title [BUG] itemized deduction ceiling switch correctly processed by taxbrain [BUG] itemized deduction ceiling switch not correctly processed by taxbrain Aug 2, 2017
@martinholmer
Copy link
Contributor

@hdoupe, Thanks for the bug report in issue #598.
Let me be sure I understand your findings.
Are you saying that when you use the regular GUI interface to TaxBrain, the reform passed to taxcalc is correct, but when you use the file-upload interface to TaxBrain, the reform passed to TaxBrain is incorrect? Is this right? Or am I confused?

@brittainhard @PeterDSteinberg

@hdoupe
Copy link
Collaborator Author

hdoupe commented Aug 2, 2017

@martinholmer Yes, it appears that there is an issue with the GUI interface to Taxbrain, but the file upload functionality appears to work correctly.

@PeterDSteinberg @brittainhard

@martinholmer
Copy link
Contributor

@hdoupe said:

it appears that there is an issue with the GUI interface to Taxbrain, but the file upload functionality appears to work correctly.

So, I had it reversed. You're finding that the GUI interface to TaxBrain has the reform wrong while the file-upload interface to TaxBrain has it right. Is that correct?

@martinholmer martinholmer changed the title [BUG] itemized deduction ceiling switch not correctly processed by taxbrain Itemized deduction ceiling switch not correctly processed by taxbrain Aug 2, 2017
@hdoupe
Copy link
Collaborator Author

hdoupe commented Aug 2, 2017

@martinholmer said

So, I had it reversed. You're finding that the GUI interface to TaxBrain has the reform wrong while the file-upload interface to TaxBrain has it right. Is that correct?

Yes, that is correct. My apologies for the confusion. I edited my initial comment in an effort to clear this up.

@martinholmer martinholmer changed the title Itemized deduction ceiling switch not correctly processed by taxbrain Itemized deduction ceiling switch not correctly processed by TaxBrain GUI interface Aug 2, 2017
@brittainhard brittainhard added this to the 1.0.2 Release milestone Aug 17, 2017
@hdoupe
Copy link
Collaborator Author

hdoupe commented Sep 28, 2017

Closed via #641

@hdoupe hdoupe closed this as completed Sep 28, 2017
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