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

Adds (then zeroes) door PSI-factors for NECBs #1734

Open
wants to merge 2 commits into
base: nrcan_421
Choose a base branch
from
Open

Conversation

brgix
Copy link
Collaborator

@brgix brgix commented Apr 25, 2024

Pull request overview

The initial implementation of TBD within BTAP (to support NECBs) applied the same fenestration interface PSI factors (W/K per linear meter) to opaque door edges. In a nutshell, 3.1.1.7. 1) of NECBs 2017 & 2020 require Ut calculations include PSI-factors from interface details (or "junctions") around "glazing assemblies", which would include OpenStudio subsurface types:

  • "FixedWindow"
  • "OperableWindow"
  • "Skylight"
  • "TubularDaylightDome"
  • "TubularDaylightDiffuser"
  • "GlassDoor".

Interface details around (opaque) "Door" and "OverheadDoor" subsurfaces would have to be excluded from BTAP's NECB 2017 and NECB2020 rulesets, as they are not included in 3.1.1.7. 1).

This came up when adding the new ASHRAE 90.1 2022 thermal bridging PSI-factor sets to TBD: ASHRAE 90.1 2022 clearly states the need to factor PSI-factors around vertical fenestration (a defined term), although it remains unclear what to do around skylights and opaque doors (see Q3 in this UnmetHours post). TBD now provides an extended thermal bridge type list, including:

  • :fenestration
  • :door
  • :skylight

... and related:

  • :balcony
  • :balconysill
  • :balconydoorsill

Solution:

  • extend (and test) revised list of thermal bridge types here, to cover all possible thermal bridge types (i.e. edge types).
  • assign (0 W/K per linear meter) to all opaque door related edge types

@brgix brgix requested a review from phylroy April 25, 2024 13:18
@brgix brgix self-assigned this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant