RoomAirModelManager::CheckEquipName function possible issues #6321
Labels
Defect
Includes code to repair a defect in EnergyPlus
MediumComplexityApproved
Used for subcontractor defect complexity requests
Issue overview
For #6272, need to eliminate use of ZoneEquipConfig.ReturnAirNode and search through all possible return nodes. Posting this issue now, but I will be working in this code, so any changes here should wait at least until #6272 is complete and merged.
Looking at this function, it is getting various zone equipment objects and assigning node names based on Alpha( n ). This is fragile if the alphas move there won't be any way to know this is broken (test file only uses PTACs). This should either use mining functions to get node numbers from the equipment, or after the JSON input processor drops in, this could grab node names using field names directly.
Also, this code for AirLoopHVACReturnAir does not look correct. Since there is no such object type, the alphas will be empty.
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.
The text was updated successfully, but these errors were encountered: