-
Notifications
You must be signed in to change notification settings - Fork 30
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
Automate All The Things #349
Conversation
Latest commit finishes the automation process. Now |
I'll leave this open for another day or two and if there are no comments I'll merge. |
If there are no objections, I'll merge this at COB today. |
This PR contributes two new bits of automation.
First, it finishes automating CBO updates. I'm going to continue working on it a bit more to try and automatically update documentation as well.
Second, it automatically generates reports detailing changes to TaxData. The new
reports.py
file will look for variables that are added/removed from both the PUF and CPS, compare CBO projections and growth factors between the master branch and whatever the current branch you're working on is, and finally compare taxcalc projections using the CPS file. It's still a work in progress, but I've attached a sample report for feedback. I'm still deciding how to best make comparisons using the PUF.Sample usage is:
"1,2,3"
is a list of PRs that have gone been/will be merged whose changes are reflected in the report. This could also be used to point to issues that are addressed. I'll update the language in the report to reflect that before my next commit.--desc sample.txt
is an optional path to a text/markdown file with additional information to be included at the start of the report. In the file used to create the sample report,sample.txt
just contained the line:I'm interested in hearing what people think of the report and what they'd like to see added/removed. The purpose of all this is so that contributors have an easy way of showing how their changes affect our datasets and taxcalc projections as well as to create a history that's easier to refer back on than digging through a bunch of PR's would be.
taxdata_report_2020-08-02.pdf