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

Add preset for Brown-Khanna GAIN Act #704

Closed
martinholmer opened this issue Oct 13, 2017 · 14 comments
Closed

Add preset for Brown-Khanna GAIN Act #704

martinholmer opened this issue Oct 13, 2017 · 14 comments
Assignees
Milestone

Comments

@martinholmer
Copy link
Contributor

TaxBrain should add a preset for the Brown-Khanna GAIN Act, which calls for a major EITC expansion.

@MattHJensen requested this in Tax-Calculator issue 1552. The JSON reform file for that reform is now available at taxcalc/reforms/BrownKhanna.json.

All that is left to do is for TaxBrain to add it to the bottom of the Presets list in the Get Started section.

screen shot 2017-10-12 at 11 20 13 am

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 13, 2017

@martinholmer This should be an easy addition for 1.1.0. Two more things are needed though:

  1. Reform - a link to the edit parameters page with reform values filled in. For example: Clinton link
  2. Notes - a google doc with a more human readable summary of the reform. For example: Clinton notes

@martinholmer
Copy link
Contributor Author

@hdoupe said in #704 that we need:

Notes - a google doc with a more human readable summary of the reform.

With respect to the Notes, can you just point to a PDF?
See the links in the original post by @MattHJensen in Tax-Calculator issue 1552.

@travelmail26
Copy link

This is great! Two notes

  1. i had issues doing the eitc because i couldnt figure out how to indicate both kink points. Khannas plan has an end to the phase in and an end to phase out. this is what should give the EITC is flattop trapezoid shape. in the bill, khanna has these points

  2. booker also released his bill with an EITC and CTC

https://www.baldwin.senate.gov/imo/media/doc/The%20Stronger%20Way%20Act%20Backgrounder.pdf

Might be worth adding so tax plans can be compared

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 13, 2017

@martinholmer said

With respect to the Notes, can you just point to a PDF?

This is my understanding of the usefulness of creating a separate notes document. The notes document is helpful because it is a human readable document that indicates which parts of the reform we were able to model and what had to be excluded. Also, a link to the original document is listed in the notes.

@hdoupe hdoupe added this to the 1.1.0 Release milestone Oct 13, 2017
@martinholmer
Copy link
Contributor Author

@hdoupe, Concerning issue #704, there is a broader problem here. Like a number of other things in the PolicyBrain application, the "Presets" section is not part of the PolicyBrain repository.

The first step in resolving #704 is to move everything having to do with the PolicyBrain application into the PolicyBrain repo, as we have discussed many times (see open issue 664 and closed issue 685, for a couple of examples of this recurring problem).

The second step is make TaxBrain read the taxcalc/reforms/*.json file of each preset reform in order to get the values of policy parameters for the TaxBrain GUI input page. There should be no duplication of the specification of the reform in the PolicyBrain repo. (Maybe this is already the way TaxBrain works, but I can't tell because the "Preset" text is not in the repo.)

This strategy will add to the repo:

  1. The "Presets" text, which now is apparently in the secret Django account
  2. The notes for each report, which now is in some (many?) Google Docs accounts.

@MattHJensen @GoFroggyRun

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 17, 2017

@martinholmer I agree with you. These documents need to be moved into the PolicyBrain repo. Currently, they are in the admin part of the Django site.

@GoFroggyRun I can give you access to these files. Would you mind moving them into the PolicyBrain repo?

@GoFroggyRun
Copy link
Contributor

GoFroggyRun commented Oct 18, 2017

@martinholmer said:

The first step in resolving #704 is to move everything having to do with the PolicyBrain application into the PolicyBrain repo, as we have discussed many times (see open issue 664 and closed issue 685, for a couple of examples of this recurring problem).

and @hdoupe replied:

I agree with you. These documents need to be moved into the PolicyBrain repo. Currently, they are in the admin part of the Django site.

+1 here.

@martinholmer also said:

The second step is make TaxBrain read the taxcalc/reforms/*.json file of each preset reform in order to get the values of policy parameters for the TaxBrain GUI input page. There should be no duplication of the specification of the reform in the PolicyBrain repo. (Maybe this is already the way TaxBrain works, but I can't tell because the "Preset" text is not in the repo.)

By looking at the website element, those reforms seem merely pre-edited TaxBrain hyper links. I agree that "there should be no duplication of the specification of the reform in the PolicyBrain repo."

@GoFroggyRun
Copy link
Contributor

@hdoupe said:

I can give you access to these files. Would you mind moving them into the PolicyBrain repo?

No problem.

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 18, 2017

@GoFroggyRun great, thanks. I just emailed the necessary info over to you so that you can get started.

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 26, 2017

With the merge of #711, we can easily add this preset. Here's a link to the edit parameter page for this run: https://www.ospc.org/taxbrain/edit/19212/?start_year=2017

@martinholmer would you mind putting together a reform notes document like this one: https://docs.google.com/document/d/1B3VhYhOtDRR5Q613vyiV236TOmqvnn29E9APYk-mll4/edit?

Upon creating this reform notes document, we can add this reform as a preset.

@martinholmer
Copy link
Contributor Author

@hdoupe said:

With the merge of #711, we can easily add this preset. Here's a link to the edit parameter page for this run: > https://www.ospc.org/taxbrain/edit/19212/?start_year=2017

@martinholmer would you mind putting together a reform notes document like this one:
https://docs.google.com/document/d/1B3VhYhOtDRR5Q613vyiV236TOmqvnn29E9APYk-mll4/edit?

Upon creating this reform notes document, we can add this reform as a preset.

It's great that #711 moved the secret HTML code into the PolicyBrain repository (thanks, @GoFroggyRun, for doing that), but the preset Notes are still not in the PolicyBrain repository. I thought we had decided to move everything out of secret places into the public repository?

@martinholmer
Copy link
Contributor Author

@hdoupe, It seems as if the Notes for each preset reform should be in a markdown document that is in the PolicyBrain repository. You can copy and paste into the PolicyBrain repo the BrownKhanna.md file included in Tax-Calculator pull request 1607.

I'll leave it up to the PolicyBrain developers to translate the Google Docs into markdown files in the PolicyBrain repository.

@hdoupe
Copy link
Collaborator

hdoupe commented Oct 27, 2017

@martinholmer Thank you for your contribution in creating the notes file. I agree with you that the current system is sub optimal, but I hesitate to make this change for two reasons.

  1. We are releasing today and time is tight. Thus, while we could just use a link to the mark-down file that you made, we do not have enough time to translate the other files before release.
  2. This is a major change from our current system. I think that we should open up an issue and discuss the best way to display these files. It may be that your solution is the one that we choose. However, we should reach a consensus before we move to a new reform notes system.

Does this sound sensible?

@martinholmer
Copy link
Contributor Author

martinholmer commented Oct 27, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants