Open erikbeeren opened 3 years ago
@erikbeeren Thanks for the bug report.
Can you please send me an example .osm file of the building? Another .osm from an HVAC system that did work should be fine.
@chriswmackey can you link to the openstudio-standards call in Honeybee that call the openstudio-standards method?
The DOASwithfancoilchillerwithbaseboardelectric is an exemple which resulted in a fatal error. The other one is the simulation of a DOASwithfancoildistrictchilledwaterwithdistricthotwater system which ran perfectly.
This bug is caused by an oversight in not testing every available system in the create typical building Model.hvac options.
It arises from a hot water loop still being added to DOAS systems when the 'air_loop_heating_type' flag in model_add_hvac_system is set to 'Water' (the default) even if the main_heat_fuel type is nil. https://github.com/NREL/openstudio-standards/pull/976/files
The fix for this is to check whether the main_heat_fuel type is nil, and ignore adding a hot water loop for DOAS coils if so.
This will be in the next release of openstudio-standards, however a quick fix is to set the 'air_loop_heating_type' to nil in the model_add_hvac_system method call, e.g.:
standard.model_add_hvac_system(self, 'DOAS', ht = nil, znht = nil, cl = 'Electricity', zones, air_loop_heating_type: nil)
Thank you, @mdahlhausen and @erikbeeren .
Here is the place where I am calling the standards gem methods on the Honeybee side of things: https://github.com/ladybug-tools/honeybee-openstudio-gem/blob/master/lib/to_openstudio/hvac/Model.hvac.rb
Essentially the honeybee-openstudio
gem includes a full copy of the Model.hvac.rb
file that's used in the "Create Typical Building" measure. So I will implement your temporary fix there, @mdahlhausen . We should also probably let @DavidGoldwasser or someone else looking after the "Create Typical Building" measure know so that they can implement the fix there.
Implemented the quick fix on the Honeybee side of things and I verified that it's all running through E+ correctly: https://github.com/ladybug-tools/honeybee-openstudio-gem/pull/150
Thanks again @mdahlhausen !
Hi @mdahlhausen and @chriswmackey, I did the same batch simulations for all ALL_AIR systems. The following systems give a fatal error:
Severe Autosizing of heating coil UA failed for Coil:Heating:Water "01_1_5EE4E735 PSZ-AC WATER HTG COIL"
And finaly I tested all Heat and Cool templates. Here all "direct evap coolers" gave a fatal error:
And the
Water source heat pumps fluid cooler with boiler
gave the folowing error:
Thanks @erikbeeren . I haven't written tests for all the HVAC systems with the new OS update - I think there were some EMS changes that break systems that rely heavily on EMS code (ASHPs, direct evap coolers). I'll look into those.
Some systems can have autosizing fail if depending on climate zone / thermal zoning. If a the design temperature delta for a coil is too small, it will cause UA sizing errors. I'm not sure of the best way to address this, though it is a longer term issue we are thinking about trying to make HVAC systems that are robust to a wide array of input models.
Hi @mdahlhausen,
Thank you for looking at al these issues. Today I also got a message from @chriswmackey that some issues where not caused by the openstudio gems. Here what Chris responded to me.
The "PSZ-AC district chilled water with baseboard district hot water" actually isn’t an error on Matt’s end. It’s an issue in the Create Typical Building measure which I used as the basis for the Honeybee HVAC templates. It seems that there’s already a "PSZ-AC district chilled water with district hot water" system template that uses baseboards with district hot water. So the "PSZ-AC district chilled water with baseboard district hot water" is completely redundant and unnecessary. I am going to remove it from the honeybee interface and SDK now. And you can tell Matt that he is off the hook for this one but that someone should update that measure at some point.
Hi @mdahlhausen
I got your name from @chriswmackey. I am using the new Ladybug tools and was testing the different HVAC templates which are based, as I understand, on the Openstudio Standard gems. I did a batch simulation with colibri/honeybee/grasshopper for all standard HB DOAS HVAC Templates. I discovered that all templates with baseboard electric, gas unit heaters and no heat resulted in a fatal error during the simulation giving me the folowing error report:
It looks like there is missing a scheme in these templates.