Skip to content

Incorrect zone index used for ZoneList in HVACManager #11323

@rraustad

Description

@rraustad

Issue overview

Review of code in HVACManager shows these lines:

for (ListNum = 1; ListNum <= state.dataHeatBal->NumOfZoneLists; ++ListNum) {
    auto &zoneList = state.dataHeatBal->ZoneList(ListNum);
    for (ZoneNum = 1; ZoneNum <= zoneList.NumOfZones; ++ZoneNum) {
        auto const &zoneSysEnergyDemand = state.dataZoneEnergyDemand->ZoneSysEnergyDemand(zoneList.Zone(ZoneNum));
        Mult = state.dataHeatBal->Zone(ZoneNum).Multiplier;

where in Mult = state.dataHeatBal->Zone(ZoneNum).Multiplier; the zone index should be the same as the zone index used in the line above as:

Mult = state.dataHeatBal->Zone(zoneList.Zone(ZoneNum)).Multiplier;

see https://github.com/NREL/EnergyPlus/blob/develop/src/EnergyPlus/HVACManager.cc#L2185-L2189

Operating System (Multiple choices)

Windows

Operating System Version

Windows 11

Version of EnergyPlus

25.2

Unmethours link or helpdesk ticket number

N/A

Defect file

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    TriageIssue needs to be assessed and labeled, further information on reported might be neededUnconfirmedDefectDefect has not yet confirmed to be an actual issue

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions