Skip to content

Rule Test JSON 5-1 update#1927

Open
jugonzal07 wants to merge 1 commit intodevelopfrom
RT/JG/ruletest_5_1_update
Open

Rule Test JSON 5-1 update#1927
jugonzal07 wants to merge 1 commit intodevelopfrom
RT/JG/ruletest_5_1_update

Conversation

@jugonzal07
Copy link
Contributor

Updated the rule test JSON based on this pull request:

https://github.com/pnnl/ruleset-checking-tool/pull/1852/changes#diff-7e5d12351a30a3e888a174ae05e77ad7e3de53bb62306ccbae018f02ede56340

NOTE: This rule test JSON is NOT part of a spreadsheet given it's unique set of orientations. This is created manually.

"unmet_occupied_load_hours_cooling": 0,
"building_peak_cooling_load": 25000
"building_peak_cooling_load": 25000,
"baseline_building_performance_energy_cost": 50000
Copy link
Collaborator

Choose a reason for hiding this comment

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

I tested this RPD against the updated 5-1 from Jackson's PR. It turned out that the RPD is invalid due to this key. I believe this key should be under the output key instead of the model_output (similar to the rule_1_2.json). Can you double-check this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I will take a look.

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.

2 participants

Comments