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

RS/YJ/Rule 11-6 (include only necessary codes) #1553

Merged
merged 13 commits into from
Dec 3, 2024
Merged
Show file tree
Hide file tree
Changes from 12 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions docs/section11/Rule11-6.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,13 @@
**Schema Version:** 0.0.37
**Mandatory Rule:** True
**Rule ID:** 11-6
**Rule Description:** Piping losses shall not be modeled.
**Rule Description:** Piping losses shall not be modeled.
**Rule Assertion:** Options are PASS/FAIL/NOT_APPLICABLE/UNDETERMINED
**Appendix G Section Reference:** Table G3.1 #11, baseline column, i

**Evaluation Context:** Each SWH Distribution System
**Data Lookup:**
**Function Call:**

1. get_swh_components_associated_with_each_swh_distribution_system()

**Applicability Checks:**
Expand All @@ -20,13 +19,13 @@
## Rule Logic:
- use get_swh_components_associated_with_each_swh_distribution_system to get the SWH BATs and SWH equipment in the building: `swh_distribution_and_eq_dict = get_swh_components_associated_with_each_swh_distribution_system(B_RMD)`
- create a value indicating whether piping losses were modeled: `piping_losses_modeled = false`
- look at every ServiceWaterPiping connected to the distribution: `for piping_id in swh_distribution_and_eq_dict[distribution_id]["Piping"]`
- look at every ServiceWaterPiping including child pipings connected to the distribution: `for piping_id in swh_distribution_and_eq_dict[distribution_id]["Piping"]`
- get the piping: `piping = get_component_by_id(piping_id, B_RMD, ServiceWaterPiping) `
- check if the piping has piping_losses_modeled: `if piping.are_thermal_losses_modeled:`
- set the piping_losses_modeled to true and go directly to rule assertion: `piping_losses_modeled = true: GO TO RULE_ASSERTION`

- **Rule Assertion - Zone:**
- Case1: piping losses are not modeled, PASS: `if !piping_losses_modeled: PASS`
- Case1: piping losses are not modeled, PASS: `if piping_losses_modeled == False: PASS`
- Case2: piping losses are modeled, FAIL: `else: FAIL`


Expand Down
2 changes: 1 addition & 1 deletion rct229/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ def cli():
Software test workflow, add sections to do test. \n
--ruleset or -rs: default is ashrae9012019, available: ashrae9012019\n
argument (optional): section string, \n
currently available: section1, section4, section5, section6, section10, section12, section16, section18, section19, section21, section22 and section23"""
currently available: section1, section4, section5, section6, section10, section11, section12, section16, section18, section19, section21, section22 and section23"""


@cli.command(
Expand Down
1 change: 1 addition & 0 deletions rct229/rulesets/ashrae9012019/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"section5",
"section6",
"section10",
"section11",
"section12",
"section16",
"section18",
Expand Down
32 changes: 32 additions & 0 deletions rct229/rulesets/ashrae9012019/section11/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# Add all available rule modules in __all__
import importlib

__all__ = [
# "section11rule1",
# "section11rule2",
# "section11rule3",
# "section11rule4",
# "section11rule5",
"section11rule6",
# "section11rule7",
# "section11rule8",
# "section11rule9",
# "section11rule10",
# "section11rule11",
# "section11rule12",
# "section11rule13",
# "section11rule14",
# "section11rule15",
# "section11rule16",
# "section11rule17",
]


def __getattr__(name):
if name in __all__:
return importlib.import_module("." + name, __name__)
raise AttributeError(f"module {__name__!r} has no attribute {name!r}")


def __dir__():
return sorted(__all__)
93 changes: 93 additions & 0 deletions rct229/rulesets/ashrae9012019/section11/section11rule6.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,93 @@
from collections import deque

from rct229.rule_engine.rule_base import RuleDefinitionBase
from rct229.rule_engine.rule_list_indexed_base import RuleDefinitionListIndexedBase
from rct229.rule_engine.ruleset_model_factory import produce_ruleset_model_description
from rct229.rulesets.ashrae9012019 import BASELINE_0
from rct229.rulesets.ashrae9012019.ruleset_functions.get_swh_equipment_associated_with_each_swh_distriubtion_system import (
get_swh_equipment_associated_with_each_swh_distribution_system,
)
from rct229.utils.jsonpath_utils import find_all


class Section11Rule6(RuleDefinitionListIndexedBase):
"""Rule 6 of ASHRAE 90.1-2019 Appendix G Section 11 (Service Water Heating)"""

def __init__(self):
super(Section11Rule6, self).__init__(
rmds_used=produce_ruleset_model_description(
USER=False, BASELINE_0=True, PROPOSED=False
),
each_rule=Section11Rule6.RMDRule(),
index_rmd=BASELINE_0,
id="11-6",
description="Piping losses shall not be modeled.",
ruleset_section_title="Service Water Heating",
standard_section="Table G3.1 #11, baseline column, i",
is_primary_rule=True,
list_path="ruleset_model_descriptions[0]",
)

class RMDRule(RuleDefinitionListIndexedBase):
def __init__(self):
super(Section11Rule6.RMDRule, self).__init__(
rmds_used=produce_ruleset_model_description(
USER=False, BASELINE_0=True, PROPOSED=False
),
each_rule=Section11Rule6.RMDRule.SWHDistRule(),
index_rmd=BASELINE_0,
list_path="$.service_water_heating_distribution_systems[*]",
)

def is_applicable(self, context, data=None):
rmd_b = context.BASELINE_0

swh_dist_sys_b = find_all(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Minor - swh_dist_sys_b to swh_dist_sys_list_b

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Addressed

"$.service_water_heating_distribution_systems[*]", rmd_b
)

return swh_dist_sys_b

def create_data(self, context, data):
rmd_b = context.BASELINE_0

swh_distribution_and_eq_list_b = (
get_swh_equipment_associated_with_each_swh_distribution_system(rmd_b)
)

return {"swh_distribution_and_eq_list_b": swh_distribution_and_eq_list_b}
Copy link
Collaborator

Choose a reason for hiding this comment

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

This data is not used in the following methods. Why do we want to create it here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch. Addressed.


class SWHDistRule(RuleDefinitionBase):
def __init__(self):
super(Section11Rule6.RMDRule.SWHDistRule, self).__init__(
rmds_used=produce_ruleset_model_description(
USER=False,
BASELINE_0=True,
PROPOSED=False,
),
required_fields={
"$": ["service_water_piping"],
},
)

def get_calc_vals(self, context, data=None):
swh_dist_sys_b = context.BASELINE_0

piping_losses_modeled_b = []
for service_water_piping in swh_dist_sys_b["service_water_piping"]:
queue = deque([service_water_piping])
while queue:
current_piping = queue.popleft()
children_piping = current_piping.get("child", [])
queue.extend(children_piping)

piping_losses_modeled_b.append(
current_piping.get("are_thermal_losses_modeled")
)

return {"piping_losses_modeled_b": piping_losses_modeled_b}

def rule_check(self, context, calc_vals=None, data=None):
piping_losses_modeled_b = calc_vals["piping_losses_modeled_b"]

return not any(piping_losses_modeled_b)
Original file line number Diff line number Diff line change
@@ -0,0 +1,143 @@
{
"rule-11-6-a": {
"Section": 11,
"Rule": 6,
"Test": "a",
"test_description": "A one story, single zone building has service hot water piping. The service hot water piping losses are correctly not modeled.",
"expected_rule_outcome": "pass",
"standard": {
"rule_id": "11-6",
"ruleset_reference": "Table G3.1 #11, baseline column, i",
"rule_description": "Piping losses shall not be modeled.",
"applicable_rmr": "Baseline Model",
"rule_assertion": "=",
"comparison_value": "Expected Value",
"mandatory_rule": "Yes",
"schema_version": "0.0.34"
},
"rmd_transformations": {
"baseline": {
"id": "ASHRAE229 1",
"calendar": {
"is_leap_year": false
},
"data_timestamp": "2024-02-12T12:00Z",
"ruleset_model_descriptions": [
{
"id": "RMD 1",
"buildings": [
{
"id": "Building 1",
"building_segments": [
{
"id": "Building Segment 1",
"zones": [
{
"id": "Thermal Zone 1",
"floor_name": "Floor1",
"spaces": [
{
"id": "Space 1",
"service_water_heating_uses": [
{
"id": "SHW 1",
"served_by_distribution_system": "SHW Distribution 1"
}
]
}
]
}
]
}
]
}
],
"service_water_heating_distribution_systems": [
{
"id": "SHW Distribution 1",
"service_water_piping": [
{
"id": "SHW Piping 1",
"are_thermal_losses_modeled": false
}
]
}
],
"type": "BASELINE_0"
}
]
}
}
},
"rule-11-6-b": {
"Section": 11,
"Rule": 6,
"Test": "b",
"test_description": "A one story, single zone building has service hot water piping. The service hot water piping losses are modeled.",
"expected_rule_outcome": "fail",
"standard": {
"rule_id": "11-6",
"ruleset_reference": "Table G3.1 #11, baseline column, i",
"rule_description": "Piping losses shall not be modeled.",
"applicable_rmr": "Baseline Model",
"rule_assertion": "=",
"comparison_value": "Expected Value",
"mandatory_rule": "Yes",
"schema_version": "0.0.34"
},
"rmd_transformations": {
"baseline": {
"id": "ASHRAE229 1",
"calendar": {
"is_leap_year": false
},
"data_timestamp": "2024-02-12T12:00Z",
"ruleset_model_descriptions": [
{
"id": "RMD 1",
"buildings": [
{
"id": "Building 1",
"building_segments": [
{
"id": "Building Segment 1",
"zones": [
{
"id": "Thermal Zone 1",
"floor_name": "Floor1",
"spaces": [
{
"id": "Space 1",
"service_water_heating_space_type": "OFFICE",
"service_water_heating_uses": [
{
"id": "SHW 1",
"served_by_distribution_system": "SHW Distribution 1"
}
]
}
]
}
]
}
]
}
],
"service_water_heating_distribution_systems": [
{
"id": "SHW Distribution 1",
"service_water_piping": [
{
"id": "SHW Piping 1",
"are_thermal_losses_modeled": true
}
]
}
],
"type": "BASELINE_0"
}
]
}
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,11 @@
"output_validation_points": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/zones[0]/terminals[0]/fan/output_validation_points",
"preheat_system": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/heating_ventilating_air_conditioning_systems[0]/preheat_system",
"pumps": "ruleset_model_descriptions[0]/pumps",
"sensible_cool_capacity": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/heating_ventilating_air_conditioning_systems[0]/cooling_system/sensible_cool_capacity",
"schedules": "ruleset_model_descriptions[0]/schedules",
"sensible_cool_capacity": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/heating_ventilating_air_conditioning_systems[0]/cooling_system/sensible_cool_capacity",
"service_water_heating_distribution_systems": "ruleset_model_descriptions[0]/service_water_heating_distribution_systems",
"service_water_piping":"ruleset_model_descriptions[0]/service_water_heating_distribution_systems[0]/service_water_piping",
"service_water_heating_uses":"ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/zones[0]/spaces[0]/service_water_heating_uses",
"spaces": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/zones[0]/spaces",
"subsurfaces": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/zones[0]/surfaces[0]/subsurfaces",
"surfaces": "ruleset_model_descriptions[0]/buildings[0]/building_segments[0]/zones[0]/surfaces",
Expand Down
6 changes: 5 additions & 1 deletion rct229/schema/ASHRAE229.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -4226,6 +4226,10 @@
"description": "Indicates if service water heating piping is a loop and recirculates",
"type": "boolean"
},
"are_thermal_losses_modeled": {
"description": "Indicates if thermal losses from the service water heating piping are modeled",
"type": "boolean"
},
"insulation_thickness": {
"description": "Pipe insulation thickness",
"type": "number",
Expand Down Expand Up @@ -5080,4 +5084,4 @@
},
"version": "0.0.36",
"$ref": "ASHRAE229.schema.json#/definitions/RulesetProjectDescription"
}
}
Loading