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?

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.

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