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-7 #1526

Draft
wants to merge 36 commits into
base: develop
Choose a base branch
from
Draft

RS/YJ/Rule 11-7 #1526

wants to merge 36 commits into from

Conversation

yunjoonjung-PNNL
Copy link
Collaborator

This rule depends on the get_swh_bats_and_swh_use function development. Hence, it isn't ready to be reviewed.

@yunjoonjung-PNNL yunjoonjung-PNNL self-assigned this Sep 16, 2024
@yunjoonjung-PNNL yunjoonjung-PNNL marked this pull request as draft September 16, 2024 23:38
@yunjoonjung-PNNL yunjoonjung-PNNL marked this pull request as ready for review September 16, 2024 23:39
@yunjoonjung-PNNL yunjoonjung-PNNL marked this pull request as draft September 16, 2024 23:39
Copy link
Collaborator

@weilixu weilixu left a comment

Choose a reason for hiding this comment

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

This is a difficult rule and I think the evaluation shall happens at SWH_BAT level, which requires customization.

Parameters
----------
building_area_type: str
building area type of the service water heating system
Copy link
Collaborator

Choose a reason for hiding this comment

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

Specify that this building_area_type is ServiceWaterHeatingSpaceOptions2019ASHRAE901

{
"Baseline_Service_Water_Heating_System": [
{
"building_area_type": "Automotive facility",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think the str would be the description of the ServiceWaterHeatingSpaceOptions2019ASHRAE901, but the enums.
So for example, the Automotive facility here, should be AUTOMOTIVE_FACILITY

rct229/rulesets/ashrae9012019/section11/section11rule7.py Outdated Show resolved Hide resolved
):
return True
else:
return False
Copy link
Collaborator

Choose a reason for hiding this comment

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

My general feeling is the RDS did not explain where the applicability/evaluation happens. The note says Applicability is determined at the building segment level, but the rule is evaluated for all SWH equipment serving the building segment. However, the logic is on swh_bat.

In that case, I suggest we use evaluate at swh_bat level.
This would require restructure of the context to only include the returned value from get_SWH_components_associated_with_each_SWH_bat. See Section5rule14 for the implementation.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Confirmed with Karen, this rule should be evaluated at building area type level - we should update the note in the RDS to reflect this change.

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.

3 participants