-
-
Notifications
You must be signed in to change notification settings - Fork 37
Update handling of tax func params for new ogcore #69
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
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #69 +/- ##
==========================================
- Coverage 78.56% 78.36% -0.20%
==========================================
Files 21 21
Lines 1376 1382 +6
==========================================
+ Hits 1081 1083 +2
- Misses 295 299 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
|
@rickecon Changes now avoid miniconda build issue and tests are passing on 3.8 on GH Actions. Not sure why there are failures on 3.7 and 3.9, but all tests pass locally on Python 3.10 for me. If you can confirm that, I think this PR is good to go. |
Remove Python 3.7 tests
|
@jdebacker. All CI tests are passing. And I updated my repo with your newest changes, updated my conda environment, and re-ran all the tests locally. I got the following one error. This looks like a weird error between Tax-Calculator and Numpy. Let me know if you have any ideas. |
|
@jdebacker. Here is my |
|
@rickecon The error you are seeing is due to using a different version of the However, recent TaxData impute the In light of this, and assuming we want to test that the |
|
@jdebacker. I just downloaded the most recent |
|
@rickecon I modified the test to work for any vintage I believe this PR can be merged. |
|
@jdebacker. Thanks for this. Looks great. Merging. |
This PR updates
calibrate.pyto handle tax function parameters that are in lists, as they are in OG-Core v 0.10.1.