Create Rule5-42.md #1537
Open
Create Rule5-42.md #1537
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KarenWGard
requested review from
JasonGlazer,
supriyagoel,
weilixu,
claperle and
JacksonJ-KC
claperle
reviewed
Nov 5, 2024
| b. Adjustment of the clear-field U-factor in accordance with Section A10.2. | ||
|
|
||
| **Rule Assertion:** B-RMR = expected value | ||
| **Appendix G Section:** Table G3.1 Section 5(a)-1 Proposed |
claperle
reviewed
Nov 5, 2024
|
|
||
| **Rule Assertion:** B-RMR = expected value | ||
| **Appendix G Section:** Table G3.1 Section 5(a)-1 Proposed | ||
| **Appendix G Section Reference:** 5(a) |
claperle
reviewed
Nov 5, 2024
|
|
||
| **Applicability Checks:** | ||
|
|
||
| 1. A project has an exterior wall |
There was a problem hiding this comment.
Thermal bridging technically applies to all types of constructions including exterior walls, below grade walls, roofs, floors.
There was a problem hiding this comment.
yes, so this could say building has an exterior wall or an exteriors roof or a below grade wall or a floor. Not sure if there would ever be a scenario where a building doesn't have even one of these.
claperle
reviewed
Nov 5, 2024
| **Applicability:** | ||
| - look at each zone in the RMD: `for zone in P_RMD....zones:` | ||
| - look at each surface in the zone: `for surface in zone.surfaces:` | ||
| - if the surface is an exterior surface, then this rule is applicable for the entire project: `if((surface.adjacent_to == "EXTERIOR") or (surface.adjacent_to == "GROUND")): UNDETERMINED` |
There was a problem hiding this comment.
This looks good because you are actually checking for any exterior surface even though the language above says that it only applies to exterior walls.
supriyagoel
approved these changes
Nov 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.