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

New economic baseline section. #61

Closed
MattHJensen opened this issue Oct 12, 2015 · 9 comments
Closed

New economic baseline section. #61

MattHJensen opened this issue Oct 12, 2015 · 9 comments

Comments

@MattHJensen
Copy link
Contributor

There should be a new section on the TaxBrain input page that gives the users access to the economic growth rate work that @Amy-Xu is conducting in #383 (previously #368). I'm hoping that @PeterDSteinberg or @talumbau take this one.

image

This feature is a bit different than others because the growth update should be applied to both the plan x and plan y calculators. Also, only one of these parameters should be changed at a time. Susan suggested a radio button, which is what I display in the sketch above.

Note that I suspect #383 will need some refactoring, which @Amy-Xu is taking care of today, before it will work seemlessly with dropq. See this comment: PSLmodels/Tax-Calculator#368 (comment).

@talumbau
Copy link
Member

So, there would be two radio button sections? One with the title "Deviation from CBO..." and then some choices, and then another with the title "Replacement for... " and then choices. Is that correct? Is the initial state of each radio button "unselected" for both? What should happen if a selection is made in both radio button areas?

@MattHJensen
Copy link
Contributor Author

So, there would be two radio button sections? One with the title "Deviation from CBO..." and then some choices, and then another with the title "Replacement for... " and then choices. Is that correct?

Yes.

Is the initial state of each radio button "unselected" for both? What should happen if a selection is made in both radio button areas?

The initial selection should be "Deviation from CBO" and the default value will be 0, so there will be no deviation from CBO.

If someone clicks the other radio button, it should automatically unselect the Deviation from CBO radio button and vice versa.

@talumbau
Copy link
Member

OK, got it.

SOS @mistakevin is the above a quick surgical strike for you?

@MattHJensen
Copy link
Contributor Author

(just to clarify, the displayed names of the parameters and all other meta data should com from the attributes in current_law_policy.json as with the other fields on the input page. )

@talumbau
Copy link
Member

@MattHJensen I think everything is clear here now except for how to populate the radio button choices. growth.json from @Amy-Xu 's PR PSLmodels/Tax-Calculator#383 shows the _factor_adjustment and _factor_target at 0.0 for both. Should we set constants in this repo for the radio button choices - e.g. 1% above CBO, 2% above CBO, 1% below CBO, 2% below CBO for growth adjustment, and then 1%, 2%, -1%, -2% real GDP growth. Something like that?

@MattHJensen
Copy link
Contributor Author

Actually I think we're not on the same page w the radio buttons. I anticipate just two selections: one choice is _factor_target and one choice is _factor_adjustment. The user uses text boxes to enter values, just like other params.

@talumbau
Copy link
Member

OK. I think I got it now. So something like this:

screen shot 2015-10-13 at 6 12 29 pm

except with a text box next to each radio button populated with the default values for _factor_adjustment and _factor_growth.

Correct?

@MattHJensen
Copy link
Contributor Author

That's right. And the 'information i' pulling in the description attribute.

@MattHJensen
Copy link
Contributor Author

Resolved and closing.

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

No branches or pull requests

2 participants