search

NREL / openstudio-common-measures-gem

Other
8 stars 15 forks source link

ChangeBuildingLocation EPW file encoding #173

Closed eringold closed 1 month ago

eringold commented 2 months ago

A weather file downloaded from climate.onebuilding.org was encoded with ISO-8859-1, causing a MalformedCSVError when loading into the OpenStudio::Weather::Epw class in ChangeBuildingLocation. This change will catch the error and backdrop to the older encoding when encountered. Test method and file included.

@DavidGoldwasser

fixes https://github.com/NREL/openstudio-common-measures-gem/issues/174

DavidGoldwasser commented 1 month ago

Thanks, I can test and merge this

DavidGoldwasser commented 1 month ago

Can't run on NREL CI, but ran measure test on local checkout, which passed.