-
Notifications
You must be signed in to change notification settings - Fork 57
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
Update to RECS 2020 data #1031
Update to RECS 2020 data #1031
Conversation
@joseph-robertson : Done. 2ca14ec |
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.
Should n_buildings_represented
in the project ymls be updated (since adding AK/HI)?
Because integration test samples change, how do / should we check for regressions?
@@ -62,8 +62,7 @@ def expected_baseline_contents(testing) | |||
'existing.osw', | |||
'existing.xml', | |||
'in.osm', | |||
'in.idf', | |||
'schedules.csv' |
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 believe we remove schedules.csv
because now we actually sample Occupants=0?
update dictionary
b9b62ff
to
54ed919
Compare
Ll/energystar climate zones
Thanks. Addressed a4bce99. I have taken a look at the regression comparisons before this commit. Increasing the unit counts will just increase the total energy. Moving forward we should be consistent after the merge. |
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.
Assuming CI comes back with ✔️, I'm good with this.
Pull Request Description
Companion PR: ResStock-Estimation #323
Transitioning characteristics to use the EIA's 2020 RECS survey final characteristics. The development effort had the following goals:
Goals of the effort:
Characteristics that use the 2020 RECS data:
Checklist
Not all may apply:
openstudio tasks.rb update_measures
has been run