NREL / openstudio-model-articulation-gem

Other
7 stars 9 forks source link

Investigate missing internal loads for stand alone retail when using space type wizard #104

Closed DavidGoldwasser closed 2 years ago

DavidGoldwasser commented 2 years ago

https://unmethours.com/question/73734/space-type-and-construction-set-wizard-no-loads-for-901-2016-retailstandalone/

It appears from screenshots that schedules come but not internal loads. See if I can replicate, and see if impacts other measures like create typical. Also test in 3.4

DavidGoldwasser commented 2 years ago

I confirmed that this is a bug, test passes, but resulting model isn't fully populated. The code to fix this is probably in the measure in ext gem. Enhanced tests here, or in ext gem. https://github.com/NREL/openstudio-extension-gem/blob/develop/lib/openstudio/extension/core/os_lib_model_generation.rb#L3399-L3570

Some tests look ok,

Here are ones that do not have loads

And here are ones that appear fine

DavidGoldwasser commented 2 years ago

In the thee tests that plug loads don't work constructions appear fine.

DavidGoldwasser commented 2 years ago

For StripMall it is finding the building type, but then failing for all of the space type

[openstudio.standards.Model] <-2> Find objects search criteria returned no results. Search criteria: {"template"=>"DOE Ref Pre-1980", "building_type"=>"RetailStripmall", "space_type"=>"Strip mall - type 1"}. Called from :/ruby/2.7.0/gems/openstudio-standards-0.2.16/lib/openstudio-standards/standards/Standards.Model.rb:1802:in `model_find_object'.

This is method that first gets the space types for a given building type get_space_types_from_building_type(building_type, template, false)

DavidGoldwasser commented 2 years ago

RetailStandalone looks very similar, again for all spaces.

[openstudio.standards.Model] <-2> Find objects search criteria returned no results. Search criteria: {"template"=>"90.1-2007", "building_type"=>"RetailStandalone", "space_type"=>"Back_Space"}. Called from :/ruby/2.7.0/gems/openstudio-standards-0.2.16/lib/openstudio-standards/standards/Standards.Model.rb:1802:in `model_find_object'.

DavidGoldwasser commented 2 years ago

Ok, get_space_types_from_building_type is in ext gem not standards, and has hard coded space types names.

https://github.com/NREL/openstudio-extension-gem/blob/develop/lib/openstudio/extension/core/os_lib_model_generation.rb#L599-L607

Maybe space type names changed for those building types. It is likly that this also impacts how create_bar tags space types and will result in similar missing loads when create_typical is run after create_bar for affected space types

DavidGoldwasser commented 2 years ago

Working on this in branch, Confirmed doesn't impact create_bar and create_typical, was related to mapping of building type name when getting space types. I'll add test in ext gem to confirm the fix before I merge it. https://github.com/NREL/openstudio-extension-gem/tree/space_type_names_fix

DavidGoldwasser commented 2 years ago

Fix is in ext gem, also found issue with use in 2004 and empty model in standards gem. https://github.com/NREL/openstudio-standards/issues/1343

Closing here, no commit needed to his repo