-
Notifications
You must be signed in to change notification settings - Fork 14
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
Allow for more flexible microdata inputs #202
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #202 +/- ##
==========================================
- Coverage 83.13% 79.50% -3.63%
==========================================
Files 23 23
Lines 1844 1947 +103
==========================================
+ Hits 1533 1548 +15
- Misses 311 399 +88
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@andersonfrailey This PR is ready for review. I'd appreciate if you could look over my changes to the API to make sure there's nothing Tax-Brain would lose from my proposed changes. |
This PR adds functionality to make it straightforward for users to use the CPS, PUF, and TMD files that Tax-Calculator is compatible with. It also preserves the ability to include a custom data file, but adds arguments to pass custom grow factors and weights with the data.