NREL / openstudio-common-measures-gem

Other
8 stars 15 forks source link

enhancements to new warning section of OpenStudio Results #26

Closed DavidGoldwasser closed 4 years ago

DavidGoldwasser commented 4 years ago

I recently added a warning section that logs in HTML warnings from upstream measures. It also adds runner.registerValues that can be used to evaluate datapoints in large scale analyzes easier. Here are few enhancements I'd like to add.

This post demonstrates condition that would exercise both items. It is a model that has space not included in thermal zone. There is a warning in forward translation about this and OpenStudio Results had warning about this in general info section for E+ floor area not matching OpenStudio. https://unmethours.com/question/45911/interior-lighting-summary-open-studio/?comment=45965#post-id-45965

DavidGoldwasser commented 4 years ago

Few other issues with this.

DavidGoldwasser commented 4 years ago
DavidGoldwasser commented 4 years ago
DavidGoldwasser commented 4 years ago

Not sure when this broke but energyPlusOutputRequests were not functioning which impacted HVAC load profile and zone conditions. Added OSW used for pre measure simulation that adds in the output variables. After 3.1 is out test this functionality in new reporting measure code. This is awkward solution with reporting measure for each test running twice.

DavidGoldwasser commented 4 years ago
DavidGoldwasser commented 4 years ago

hvac_load_ip

This issue is related to this https://github.com/NREL/OpenStudio/issues/4092

DavidGoldwasser commented 4 years ago