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
ComStock! #65
ComStock! #65
Conversation
…itten when specified by the user in the yml file.
@nmerket FYI - I'm testing this in the next few days, so please don't worry too much about code review just yet. Cheers, |
Make sure the custom docker image on the local docker version. |
When I try to run a precomputed sample like so: baseline:
sampling_algorithm: precomputed
# n_datapoints: 12
n_buildings_represented: 110000000
precomputed_sample: ../../buildstock.csv I get the following error:
Do I need to include |
But it runs if I look at the buildstock.csv and count lines and put that in for |
Then when I provide
Based on that we should definitely let |
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.
@rHorsey I've completed testing of this on local, Eagle, and AWS. It's working as far as I can tell now. I fixed some things in the validation and simplified the pre-computed sampler. Let me know when the docs changes are done and we'll merge it.
PR to enable Commercial use of modern-day buildstockbatch.