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

Building Energy Standards Envelope Requirements Data Update #1795

Merged
merged 43 commits into from
Oct 7, 2024

Conversation

leijerry888
Copy link
Collaborator

Updates have been made to the Building Energy Standards Data repository, see here. This pull request was automatically generated.

@lymereJ lymereJ changed the title Building Energy Standards Data Update Building Energy Standards Envelope Requirements Data Update Aug 21, 2024
@lymereJ lymereJ marked this pull request as draft September 3, 2024 19:21
@lymereJ lymereJ marked this pull request as ready for review September 11, 2024 16:59
@lymereJ
Copy link
Collaborator

lymereJ commented Sep 11, 2024

@mdahlhausen - This PR is ready for your review. The only diffs are regression/performance related. I spot-checked them and they are expected.

@lymereJ lymereJ requested a review from mdahlhausen September 11, 2024 17:30
Copy link
Collaborator

@mdahlhausen mdahlhausen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes:

  • include 'ClimateZone 0' construction properties, at least for ASHRAE 90.1-2016 and 2019, which include it. The construction .json updates are removing the existing data.
  • 90.1-2013 and 90.1-2016 use 'Metal framing (all other)' in window construction properties, but should be split into 'Nonmetal framing, all', 'Metal framing, fixed', 'Metal framing, operable', and 'Metal framing, entrance door'. This relates to Revise fenestration categories #958.
  • I see you cut off the 'Window' name from exterior window construction property names. E.g. "U 1.22 SHGC 0.54 Simple Glazing Window" vs. "U 1.22 SHGC 0.54 Simple Glazing". That may be ok. I added Window a while ago to avoid confusion with the material of the same name. If you'd prefer to leave 'Window', I can adapt, but it may be nice to include. Thoughts?

Once the changes are done, I can regenerate the regression models. I'll also add the missing fields to the non-90.1 template construction properties .jsons.

@lymereJ
Copy link
Collaborator

lymereJ commented Sep 11, 2024

@mdahlhausen - ClimateZone 0 construction properties are included in this PR.

@mdahlhausen
Copy link
Collaborator

mdahlhausen commented Sep 11, 2024

@mdahlhausen - ClimateZone 0 construction properties are included in this PR.

oh my bad. I see they are, included after ClimateZone 8. I may move them to the start of the .json for consistency.
Could you update the .json export in your standards data repo to place ClimateZone 0 first?

@lymereJ
Copy link
Collaborator

lymereJ commented Sep 11, 2024

  • 90.1-2013 and 90.1-2016 use 'Metal framing (all other)' in window construction properties, but should be split into 'Nonmetal framing, all', 'Metal framing, fixed', 'Metal framing, operable', and 'Metal framing, entrance door'. This relates to Revise fenestration categories #958.

The PR actually do that split, but use a slightly different nomenclature. We can change it in the database.

@lymereJ
Copy link
Collaborator

lymereJ commented Sep 11, 2024

  • I see you cut off the 'Window' name from exterior window construction property names. E.g. "U 1.22 SHGC 0.54 Simple Glazing Window" vs. "U 1.22 SHGC 0.54 Simple Glazing". That may be ok. I added Window a while ago to avoid confusion with the material of the same name. If you'd prefer to leave 'Window', I can adapt, but it may be nice to include. Thoughts?

We can roll that back if you feel strongly about it. Our thought was that it would perhaps avoid having multiple identical constructions. For example: U 1.22 SHGC 0.54 Simple Glazing Window, U 1.22 SHGC 0.54 Simple Glazing Skylight, and U 1.22 SHGC 0.54 Simple Glazing Door would all use a material named U 1.22 SHGC 0.54 Simple Glazing.

Move climate zone 0 to the front of the construction properties .jsons for ASHRAE 90.1 versions
remove all window constructions, and all but the one used simple glazing construction
Omit 'Window', 'Window Weighted', and 'Skylight' from 'Simple Glazing' constructions
@mdahlhausen
Copy link
Collaborator

@lymereJ Thanks for the responses. I moved the climate zone 0 data up to the top of the construction properties .jsons. You may need short line of code to export in that order from the database.

I agree with your approach to simplify constructions. I removed any references to 'Simple Glazing Window' and 'Simple Glazing Skylight' and replaced them with 'Simple Glazing'.

Can you explain the standards construction type in 90.1-2013 and 90.1-2016? I see 'Metal framing (all other)' listed in the .jsons, but that doesn't correspond to types in the standard.

@lymereJ
Copy link
Collaborator

lymereJ commented Sep 16, 2024

Thanks! We're making changes to the database and will "re-sync" the data from this PR with it. It should clarify the 90.1-2013/6 terminology differences. See pnnl/building-energy-standards-data#96 for updates.

@lymereJ
Copy link
Collaborator

lymereJ commented Oct 7, 2024

@mdahlhausen - Looks like the tests are passing!

@mdahlhausen mdahlhausen self-requested a review October 7, 2024 16:24
Copy link
Collaborator

@mdahlhausen mdahlhausen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the update!

@mdahlhausen mdahlhausen merged commit 9b73867 into master Oct 7, 2024
1 of 2 checks passed
@mdahlhausen mdahlhausen deleted the data_update_0546f80 branch October 7, 2024 16:25
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

Successfully merging this pull request may close these issues.

3 participants