-
Notifications
You must be signed in to change notification settings - Fork 7
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
Are you sure you spelled 'self_employment_income_last_year' correctly? #8
Comments
Thanks - we should change these lines: policyengine-taxsim/TaxsimInputReader.py Lines 83 to 84 in 682195e
For the |
I ran the pip install and got
TypeError: __init__() got an unexpected keyword argument 'reform'
Does the emulator try to call taxsim? You should consult me about that,
because I can make taxsim into a DLL for you. I did it once before,
shouldn't be too hard. But I didn't expect you to be calling taxsim so
soon, I thought to start out with the program would just do your
calculator.
Daniel Feenberg
…On Sun, 22 Sep 2024, Max Ghenis wrote:
Thanks - we should change these lines:
https://github.com/PolicyEngine/policyengine-taxsim/blob/682195e3e36328b431896e96f0dbf9145b7945e8/TaxsimInputReader.py#L83-L84
For the `hi_income_tax` piece, we do have `hi_income_tax` in policyengine-us. Could you try updating? `pip install -U policyengine-us`
--
Reply to this email directly or view it on GitHub:
#8 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
No I don't think it calls taxsim. |
We checked in this morning and it seemed that this could be due to Python 3.9, which PolicyEngine no longer support. @feenberg will report back after updating to 3.10, which might be a couple weeks as it's a server. |
This shouldn't be an issue anymore using the package (we've removed self-employment income for now from inputs). Dan could you check? |
I just cloned policyengine-taxsim today and tried it out. Running the sample data I receive this error message:
If I remove all the data items from psemp on, then the error message complains about idtl.
If I then add idtl with value 2, I get the message
I notice that policyengine-taxsim accepts empty fields as zero. This is a good practice and I will add it to Taxsim.
The text was updated successfully, but these errors were encountered: