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

AirLoopHVAC:DedicatedOutoorAirSystem breaks unless AirLoopHVACs are all upper case. #8581

Closed
3 tasks
jmarrec opened this issue Mar 3, 2021 · 0 comments · Fixed by #8582
Closed
3 tasks
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@jmarrec
Copy link
Contributor

jmarrec commented Mar 3, 2021

Issue overview

Take SmallOffice_CentralDOAS.idf

  AirLoopHVAC:DedicatedOutdoorAirSystem,
    AirLoopHVAC DOAS,        !- Name
    AirLoopDOAS OA system,   !- AirLoopHVAC:OutdoorAirSystem Name
    ALWAYS_ON,               !- Availability Schedule Name
    AirLoopDOASMixer,        !- AirLoopHVAC:Mixer Name
    AirLoopDOASSplitter,     !- AirLoopHVAC:Splitter Name
    4.5,                     !- Preheat Design Temperature {C}
    0.004,                   !- Preheat Design Humidity Ratio {kgWater/kgDryAir}
    17.5,                    !- Precool Design Temperature {C}
    0.012,                   !- Precool Design Humidity Ratio {kgWater/kgDryAir}
    5,                       !- Number of AirLoopHVAC
-    PSZ-AC:1,                !- AirLoopHVAC 1 Name
+    PSZ-Ac:1,                !- AirLoopHVAC 1 Name
    PSZ-AC:2,                !- AirLoopHVAC 2 Name
    PSZ-AC:3,                !- AirLoopHVAC 3 Name
    PSZ-AC:4,                !- AirLoopHVAC 4 Name
    PSZ-AC:5;                !- AirLoopHVAC 5 Name

eplusout.err

   ** Severe  ** AirLoopHVAC:DedicatedOutdoorAirSystem, "AIRLOOPHVAC DOAS" AirLoopHVAC Name not found: PSZ-Ac:1
   **  Fatal  ** getAirLoopHVACDOAS: Previous errors cause termination.
   ...Summary of Errors that led to program termination:
   ..... Reference severe error count=1
   ..... Last severe error=AirLoopHVAC:DedicatedOutdoorAirSystem, "AIRLOOPHVAC DOAS" AirLoopHVAC Name not found: PSZ-Ac:

Issue is here, missing MakeUPPERCase:

std::string name = AirLoopHAVCName.at("airloophvac_name");

Details

Some additional details for this issue (if relevant):

Checklist

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

  • Defect file added (list location of defect file here)
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@jmarrec jmarrec added the Defect Includes code to repair a defect in EnergyPlus label Mar 3, 2021
@jmarrec jmarrec added this to the EnergyPlus 9.5.0 milestone Mar 3, 2021
@jmarrec jmarrec self-assigned this Mar 3, 2021
jmarrec added a commit that referenced this issue Mar 3, 2021
jmarrec added a commit that referenced this issue Mar 3, 2021
Myoldmopar added a commit that referenced this issue Mar 5, 2021
HotFix #8581: make AirLoopHVAC:DedicatedOutdoorAirSystem's airloophvac_name extension case-insentive
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
Projects
None yet
1 participant