Skip to content

Conversation

@yunjoonjung-PNNL
Copy link
Collaborator

I haven't tested this rule yet.

@yunjoonjung-PNNL yunjoonjung-PNNL marked this pull request as draft October 28, 2025 16:59
@weilixu
Copy link
Collaborator

weilixu commented Nov 4, 2025

Dependency on #1846

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.

Left comments to address

"$.buildings[*].building_segments[*].zones[*].spaces[*].miscellaneous_equipment[*]",
rmd_p,
)
][0],
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why there is [0] at the end of the list? Also, the sum here will just flatten this 2D array, instead of sum it into a value.

I think this schedule_eflh_p shall be a dictionary, in which the key is the schedule ID and the value is the sum of the hourly value?

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 point. Addressed

loads_included_p = (
misc_equip_p["power"] > 0 * ureg("W")
and misc_equip_p["sensible_fraction"] > 0
and misc_equip_p["latent_fraction"] > 0
Copy link
Collaborator

Choose a reason for hiding this comment

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

In RDS, it says: and (misc_equipment_p.sensible_fraction > 0 or misc_equipment_p.latent_fraction > 0)

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.

@weilixu
Copy link
Collaborator

weilixu commented Nov 18, 2025

Is this ready for review? @yunjoonjung-PNNL

@yunjoonjung-PNNL
Copy link
Collaborator Author

Is this ready for review? @yunjoonjung-PNNL

@weilixu - No not yet because I need to test the rule, but I've got an error running the run_ruletests.py script. I'll ask the issue during our standup today and finalize this rule.

@yunjoonjung-PNNL yunjoonjung-PNNL marked this pull request as ready for review November 25, 2025 00:10
@yunjoonjung-PNNL
Copy link
Collaborator Author

@weilixu - This rule has passed the testing JSON files from #1824.

@weilixu weilixu merged commit 56e5efa into feature/ashrae-9012022 Dec 2, 2025
5 checks passed
@weilixu weilixu deleted the RS/YJ/Rule_12-5 branch December 2, 2025 18:47
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