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

Testing weight-deviation penalty when we reweight #114

Open
donboyd5 opened this issue Jul 2, 2024 · 1 comment
Open

Testing weight-deviation penalty when we reweight #114

donboyd5 opened this issue Jul 2, 2024 · 1 comment

Comments

@donboyd5
Copy link
Collaborator

donboyd5 commented Jul 2, 2024

@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.

image

@martinholmer
Copy link
Collaborator

@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.

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