-
Notifications
You must be signed in to change notification settings - Fork 0
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 script to sample a single location #70
Conversation
Minimum allowed coverage is Generated by 🐒 cobertura-action against ab66cdd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! just added a few comments for clarity
path/to/output_dir/buildstock_G1900030_G19001800_200.csv with 200 samples | ||
|
||
Methodology: | ||
This modifies the conditional probability distributions from the standard ResStock national project |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you explain this a bit further? if we were to use the same number of samples as there is in the national resstock (i.e., county, puma in the 550k sample), that does not change the probability distributions, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated this - let me know if it's still unclear.
This script runs the residential quota sampler for a single county+PUMA.
Note: We probably won't try to merge this into the upstream
NREL/buildstockbatch
repo, so I'm saving it to RA'sdevelop
branch, rather than thegcp
branch where our other changes are.Potential future work: