-
Notifications
You must be signed in to change notification settings - Fork 32
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
Known issues with "Quick Calculation" #306
Comments
Re-opening until item 1 from above is resolved as well. |
@talumbau, As part of this pull request, can you change DataFrame.sample() random_state to 80 unless the one being used now produces sub-sample results that are even closer to full-sample results than a relative difference of 0.006 (that is, 0.6 percent) as described in Tax-Calculator pull request #869? |
@MattHJensen, What is the status of TaxBrain "Quick Calculation" issue #306? |
@martinholmer The quick calculation button is not available on TaxBrain presets, but it is available and working fine from the default input page.
Yes, although it would be helpful to add the caveat that it is not available for preset reforms, as some of users may work off of presets. |
@MattHJensen said:
Yes, I guess it would be helpful in the short run, but since the goal is to make it eventually work for preset reforms, I think I'll leave that transitory detail out of the user documentation. |
Ok, that makes a lot of sense to plan w/ the maintenance burden in mind. |
As of August 2, 2017, the first bug item mentioned nearly a year ago in webapp-public issue #306 is still not fixed. During the year since #306 was opened, the contents of the Where do these two remaining TaxBrain Quick-Calculation bug-fixes stand in our priorities? |
The text was updated successfully, but these errors were encountered: