Fix UnmetLoads measure for OS 3.0 and newer

[shorowit]

Fixes https://github.com/NREL/openstudio-common-measures-gem/issues/76

[shorowit]

@DavidGoldwasser I made the change and re-enabled the tests, but two of the tests are failing. They seem unrelated to my change, so I will let you decide what to do about them. I'm not familiar enough with the measure to know what's going on.

[aaron-boranian]

@DavidGoldwasser @shorowit Just came across this issue with Unmet Load Hours troubleshooting measure while prepping for an OpenStudio Application training workshop. In the meantime, I can create a custom version of this measure to share with workshop attendees.

It looks like Scott's pull request had some merge conflicts to resolve.

[DavidGoldwasser]

@aaron-boranian Thanks for pinging this. I'll look at merge conflicts and see if I can get tests passing and this merged

[DavidGoldwasser]

Not sure why I'm getting failures or errors in 7 measures when only one was changed. Try CI run of clean develop see what is happening, and run locally.

[DavidGoldwasser]

I started to look at this hoping to get in 3.9.0 but not sure if I'll get it in time. First I remember whey the test was commented out, it overwrites some OpenStudio methods like OpenStudio::SqlFile that break other measures that use that method that run after this.

This was one of first reporting measures and was using pre-made SQL file which was fine at the time, but now all reporting measures run a simulation and make a fresh SQL file as part of the test. I'll update this measure to do that but may not have time to do it in time to release with update for OpenStudio 3.9.0.

Another thing I noticed is the test SQL file results in 0 unmet hours which makes it hard to see much. That may be because it is old SQL file not being read correctly. Maybe will have more populated HTML with a new model.