Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zone Equipment array is not always initialized (causes exception/crash) #10459

Closed
tanaya-mankad opened this issue Apr 2, 2024 · 0 comments · Fixed by #10482
Closed

Zone Equipment array is not always initialized (causes exception/crash) #10459

tanaya-mankad opened this issue Apr 2, 2024 · 0 comments · Fixed by #10482
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus LowComplexityApproved Used for subcontractor defect complexity requests

Comments

@tanaya-mankad
Copy link
Collaborator

tanaya-mankad commented Apr 2, 2024

Issue overview

The zone equipment array (state.dataZoneEquip->ZoneEquipConfig) is not consistently initialized before its use.
Paths through the code that calculate internal heat gains, for example, initialize zone equipment there, so subsequent iterations over zone equipment in other classes such as VRF "accidentally" work. When internal heat gains (e.g. lights) are omitted from such VRF-containing models, the array is unsized and throws an exception.

Details

Some additional details for this issue (if relevant):

@tanaya-mankad tanaya-mankad added the Defect Includes code to repair a defect in EnergyPlus label Apr 2, 2024
@RKStrand RKStrand self-assigned this Apr 16, 2024
@Myoldmopar Myoldmopar added the LowComplexityApproved Used for subcontractor defect complexity requests label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus LowComplexityApproved Used for subcontractor defect complexity requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants