-
Notifications
You must be signed in to change notification settings - Fork 6
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
Testing weight-deviation penalty when we reweight #114
Comments
@donboyd5, Given the results of PR #148, there seems to be less reason to experiment with a positive value of the REWEIGHT_DEVIATION_PENALTY. But, after PR #148 is merged, it would be interesting to do this experiment: use the master branch and make one change (changing the PENALTY value from 0.0 to 0.2) and use the method illustrated in issue #146 to see how the 2023 tax expenditure estimates change when imposing the penalty. |
@nikhilwoodruff, FYI @martinholmer
Reminder that we'll want to start imposing a weight-deviation penalty in the reweighting algorithm. My last successful run of make data, using current master, has this for s006 and s006_original -- largest weight before reweighting was 9675 and after reweighting largest was 70664, which is dramatic and I think we want to prevent it. I know we've got tighter bounds about to be implemented on weight ratios, which will probably solve the problem so we can discuss impact of that first. Simply opening this issue to make sure we put some effort into analyzing and addressing this as we think best.
The text was updated successfully, but these errors were encountered: