urbanopt / urbanopt-cli

URBANopt command line interface
https://docs.urbanopt.net/
Other
6 stars 2 forks source link

Error with EMS sensor names in BuildResidentialModel measure #450

Closed JingWang-CUB closed 9 months ago

JingWang-CUB commented 9 months ago

Expected behavior

EMS sensor names for each individual unit should be initialized correctly before they are called in the EMS programs.

Actual behavior

Wrong sensor names were used in the EMS programs causing "sensor not initialized" errors for residential buildings with more than one units. image This is potentially caused by line 468 in the measure: image

To Reproduce

Steps to reproduce the behavior:

Run a Single Family Attached building with more than one units in UO CLI. The heating system type can be "residential - boiler with no cooling". UO version is 0.10.0 with OS-HPXML 1.6.0, but this should still exist in the latest UO version.

Additional context

Windows 10.

kflemin commented 9 months ago

@joseph-robertson

JingWang-CUB commented 9 months ago

Upgrading to UO 0.11.1 solved the problem.