Skip to content

Material:RoofVegetation with anything other than CTF should be fatal. #10871

@mjwitte

Description

@mjwitte

Issue overview

A user file with Material:Roofvegetation and HeatBalanceAlgorithm ConductionFiniteDifference results in unrealistic roof surface temperatures.

Looking at output variable BLOCK1:ZONE1_ROOF_1_0_0:CondFD Surface Temperature Node 1 [C](Hourly), with v9.4 it shows very high temperatures (>300C), and with v25.1-9c7c2a6d79, that node reports 0C all year.

There is a severe error message, but this should terminate the simulation since it produces garbage results.
** Severe ** CalcEcoRoof: EcoRoof simulation but HeatBalanceAlgorithm is not ConductionTransferFunction(CTF). EcoRoof model currently works only with CTF heat balance solution algorithm.

  1. Fatal out for this condition.
  2. Make sure this works ok if the file uses SurfaceProperty:HeatTransferAlgorithm or SurfaceProperty:HeatTransferAlgorithm:MultipleSurface.

Details

Some additional details for this issue (if relevant):

  • Version of EnergyPlus 25.1-9c7c2a6d79
  • Helpdesk ticket number 16816.

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • Defect file added EnergyPlusDevSupport\DefectFiles
  • Ticket added to EnergyPlus Defect Complexity (Github Project)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)

Metadata

Metadata

Assignees

Labels

DefectIncludes code to repair a defect in EnergyPlus

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions