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

Node duplication in some of the AirflowNetwork example files #7762

Closed
1 of 3 tasks
jasondegraw opened this issue Feb 11, 2020 · 6 comments · Fixed by #7798
Closed
1 of 3 tasks

Node duplication in some of the AirflowNetwork example files #7762

jasondegraw opened this issue Feb 11, 2020 · 6 comments · Fixed by #7798
Assignees
Labels
AirflowNetwork Related primarily on airflow-network portions of the codebase Defect Includes code to repair a defect in EnergyPlus

Comments

@jasondegraw
Copy link
Member

jasondegraw commented Feb 11, 2020

Issue overview

Some of the the AirflowNetwork example files have duplicate nodes: multiple AFN nodes are pointing toward the same feature/component. See, for example, AirflowNetwork_Attic_Duct.idf, where MainInletNode and FanInletNode both reference Air Loop Inlet Node. The example files still run successfully, but the networks are possibly more complex than necessary. One possible solution is to upgrade the auditing script to warn about this, but it would also work to just check all of the files to make sure that the networks represent what is intended.

Checklist

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

  • Defect file added (example file AirflowNetwork_Attic_Duct.idf)
  • 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)
@jasondegraw jasondegraw added Defect Includes code to repair a defect in EnergyPlus AirflowNetwork Related primarily on airflow-network portions of the codebase labels Feb 11, 2020
@mldichter
Copy link

mldichter commented Feb 11, 2020

I discovered the difficult AirflowNetwork layout to interpret in the AirflowNetwork_Attic_Duct.idf example file which is discussed in this UnmetHours post.
https://unmethours.com/question/42733/example-airflownetwork-drawing-for-idf/

AirflowNetwork:Distribution:Node,
    MainInletNode,           !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    1;                       !- Node Height {m}

AirflowNetwork:Distribution:Node,
    FanInletNode,            !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    1;                       !- Node Height {m}

AirflowNetwork:Distribution:Linkage,
    SystemInletLink,         !- Name
    MainInletNode,           !- Node 1 Name
    FanInletNode,            !- Node 2 Name
    MainReturn,              !- Component Name
    Occupied Thermal Zone;   !- Thermal Zone Name

@mldichter
Copy link

AirflowNetwork_MultiZone_House_OvercoolDehumid.idf
"Air Loop Inlet Node" referenced twice and used on the same linkage.

AirflowNetwork:Distribution:Node,
    MainInletNode,           !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    3.0;                     !- Node Height {m}

AirflowNetwork:Distribution:Node,
    CoolingInletNode,        !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    3.0;                     !- Node Height {m}

AirflowNetwork:Distribution:Linkage,
    SystemInletLink,         !- Name
    MainInletNode,           !- Node 1 Name
    CoolingInletNode,        !- Node 2 Name
    MainReturn,              !- Component Name
    GARAGE ZONE;             !- Thermal Zone Name

@mldichter
Copy link

mldichter commented Feb 14, 2020

AirflowNetwork_MultiZone_House_FanModel.idf
AirflowNetwork_MultiZone_House.idf
AirflowNetwork_Simple_House.idf
AirflowNetwork_MultiZone_LocalNode.idf
AirflowNetwork_MultiZone_House_TwoSpeed.idf
"Air Loop Inlet Node" referenced twice and used on the same linkage.

AirflowNetwork:Distribution:Node,
    MainInletNode,           !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    3.0;                     !- Node Height {m}

AirflowNetwork:Distribution:Node,
    FanInletNode,            !- Name
    Air Loop Inlet Node,     !- Component Name or Node Name
    Other,                   !- Component Object Type or Node Type
    3.0;                     !- Node Height {m}

AirflowNetwork:Distribution:Linkage,
    SystemInletLink,         !- Name
    MainInletNode,           !- Node 1 Name
    FanInletNode,            !- Node 2 Name
    MainReturn,              !- Component Name
    GARAGE ZONE;             !- Thermal Zone Name

@mldichter
Copy link

I went through all the energyplus example files with AirflowNetwork:Distribution:Node objects,

AirflowNetwork_Attic_Duct.idf
AirflowNetwork_MultiAirLoops.idf
AirflowNetwork_MultiZone_House.idf
AirflowNetwork_MultiZone_House_FanModel.idf
AirflowNetwork_MultiZone_House_OvercoolDehumid.idf
AirflowNetwork_MultiZone_House_TwoSpeed.idf
AirflowNetwork_MultiZone_LocalNode.idf
AirflowNetwork_MultiZone_SmallOffice.idf
AirflowNetwork_MultiZone_SmallOffice_CoilHXAssistedDX.idf
AirflowNetwork_MultiZone_SmallOffice_GenericContam.idf
AirflowNetwork_MultiZone_SmallOffice_HeatRecoveryHXSL.idf
AirflowNetwork_MultiZone_SmallOffice_VAV.idf
AirflowNetwork_PressureControl.idf
AirflowNetwork_Simple_House.idf
AirflowNetwork_Simple_SmallOffice.idf
EMSAirflowNetworkOpeningControlByHumidity.idf
HybridVentilationControl.idf

and the files in the previous comments had two AFN nodes referencing the same AirLoop node.

@lgu1234
Copy link
Contributor

lgu1234 commented Feb 19, 2020

@mldichter @jasondegraw I changed an object from

AirflowNetwork:Distribution:Node,
MainInletNode, !- Name
Air Loop Inlet Node, !- Component Name or Node Name
Other, !- Component Object Type or Node Type
3.0; !- Node Height {m}

To:

AirflowNetwork:Distribution:Node,
MainInletNode, !- Name
, !- Component Name or Node Name
Other, !- Component Object Type or Node Type
3.0; !- Node Height {m}

by removing duplicated "Air Loop Inlet Node" in the following 7 example files:

AirflowNetwork_Attic_Duct.idf
AirflowNetwork_MultiZone_House.idf
AirflowNetwork_MultiZone_House_FanModel.idf
AirflowNetwork_MultiZone_House_OvercoolDehumid.idf
AirflowNetwork_MultiZone_House_TwoSpeed.idf
AirflowNetwork_MultiZone_LocalNode.idf
AirflowNetwork_Simple_House.idf

No differences of simulation results were found. Here is my suggestion to fix this issue.

  1. Set blank for the Component Name or Node Name field for MainInletNode node.
  2. Revised the code to check the inputs of the Component Name or Node Name field to avoid duplication.

Please let me know what you think.

@jasondegraw
Copy link
Member Author

@lgu1234 That's sort of what I had expected. The information that the pressure network is passing back to the rest of the code isn't really changing the computations that EnergyPlus does. Your plan sounds good to me.

@lgu1234 lgu1234 self-assigned this Feb 20, 2020
@mitchute mitchute linked a pull request Mar 10, 2020 that will close this issue
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AirflowNetwork Related primarily on airflow-network portions of the codebase Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants