NREL / openstudio-common-measures-gem

Other
8 stars 15 forks source link

ChangeBuildingLocation CSV encoding #174

Closed MatthewSteen closed 1 month ago

MatthewSteen commented 2 months ago

The ChangeBuildingLocation measure's epw.rb module uses Rubys CSV library to read an EPW file, which can fail with some encodings.

https://github.com/NREL/openstudio-common-measures-gem/blob/6f3a8184cb0d50e3ae793be4129d895f736fd241/lib/measures/ChangeBuildingLocation/resources/epw.rb#L130

ERROR] [openstudio.measure.OSRunner] SWIG director method error. CSV::MalformedCSVError: Invalid byte sequence in UTF-8 in line 1.

Reported by Autodesk and fixed in https://github.com/NREL/gbxml-to-openstudio/pull/185.