NREL / gbxml-to-openstudio

Releases of the OpenStudio CLI for Revit Systems Analysis.
5 stars 2 forks source link

Updated import_gbxml to handle buildings with no name #113

Closed axelstudios closed 2 years ago

axelstudios commented 2 years ago

This PR fixes an issue from #109 where gbXMLs without a building name would fail with Optional not initialized. This change restores the default behavior from OpenStudio v2.8.1.

To Test

Use one of the ~15 regression gbXML files missing a building name (e.g. Clerestory.xml or Office.xml)

MatthewSteen commented 1 year ago

Here's the corresponding Autodesk PR. https://git.autodesk.com/GBS/InsightStudioMeasures/pull/18