search

NREL / openstudio-calibration-gem

Other
5 stars 5 forks source link

Update extension version #26

Closed nllong closed 3 years ago

nllong commented 3 years ago

Merge this into remove_zone_report first.

nllong commented 3 years ago

@DavidGoldwasser -- can you check this over? I ran tests in the OpenStudio 3.1.1-alpha and they all passed.

DavidGoldwasser commented 3 years ago

@nllong the only thing that seemed odd is that the readme.md which is updated by measure manager no longer thinks the measure has any arguments. That didn't happen on any other measures. Maybe there is something in the arguments section of that measure that didn't work with Ruby upgrade?

Screen Shot 2021-03-04 at 5 26 22 PM

DavidGoldwasser commented 3 years ago

Ah, when I look at XML, which was hidden until I expanded it I see this error. I think this is old error related to reporting measures having access to model and just has not been hit because has not been updated or used in GUI in a while. So this should be fixed. If I get a chance I can try to update it later tonight.

<error>wrong number of arguments (given 1, expected 0)</error>