90.1 PRM: fan power additional properties only populated for zones on AirLoops

[eringold]

@dmaddoxwhite @lymereJ @weilixu Another issue trying to use model_create_prm_stable_baseline_building on an actual model.

Fan power additional properties are set for zones served by AirLoopHVACs in the proposed model:

openstudio-standards/lib/openstudio-standards/standards/Standards.Model.rb

Lines 242 to 256 in d332605

	if model_get_fan_power_breakdown
	model.getAirLoopHVACs.sort.each do |air_loop|
	supply_fan_w = air_loop_hvac_get_supply_fan_power(air_loop)
	return_fan_w = air_loop_hvac_get_return_fan_power(air_loop)
	relief_fan_w = air_loop_hvac_get_relief_fan_power(air_loop)
	# Save fan power at the zone to determining
	# baseline fan power
	air_loop.thermalZones.sort.each do |zone|
	zone.additionalProperties.setFeature('supply_fan_w', supply_fan_w.to_f)
	zone.additionalProperties.setFeature('return_fan_w', return_fan_w.to_f)
	zone.additionalProperties.setFeature('relief_fan_w', relief_fan_w.to_f)
	end
	end
	end

After baseline air systems are made, those properties are queried:

openstudio-standards/lib/openstudio-standards/standards/ashrae_90_1_prm/ashrae_90_1_prm.AirLoopHVAC.rb

Lines 269 to 271 in d332605

	supply_fan_power_fraction += zone_air_flow * zone.additionalProperties.getFeatureAsDouble('supply_fan_w').get
	return_fan_power_fraction += zone_air_flow * zone.additionalProperties.getFeatureAsDouble('return_fan_w').get
	relief_fan_power_fraction += zone_air_flow * zone.additionalProperties.getFeatureAsDouble('relief_fan_w').get

For zones that aren't served by AirLoops in the proposed model, but end up added to one in the Baseline model, those additional properties don't exist, and because zone.additionalProperties.getFeatureAsDouble isn't 'checked', they will throw 'optional is not initialized' errors. I believe such an occurance is very likely with actual proposed models (e.g. zones served by FCUs, but that don't have different enough loads to kick them into their own single-zone systems in the baseline).

Suggest querying and adding additional properties for more HVAC types in the proposed, or handling missing additional properties within air_loop_hvac_apply_prm_baseline_fan_power.

[lymereJ]

Thanks for reporting that issue! I'll push some changes to address it. Would you be able to share a model that we could use for testing this defect?

[eringold]

@lymereJ possibly. It's kind of an unwieldy model, probably not great to have as part of a test suite. Once I'm past this deadline I can see if I can mock something up that would be lighter.

This is also my first (and rushed) attempt at using the new PRM template, so it's possible there are things about how it expects the model to be setup that I haven't fully grasped.

[lymereJ]

Sure, thanks!

If you want to share the rough model for now that'd be great, we could use it to test this fix internally and we could integrate the "better" model later.