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

AirTerminal:SingleDuct:UserDefined cannot find matching outlet node in ZoneHVAC:AirDistributionUnit #10507

Open
1 of 3 tasks
rraustad opened this issue May 16, 2024 · 4 comments
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@rraustad
Copy link
Contributor

rraustad commented May 16, 2024

Issue overview

User file shows a severe warning that appears to be a result of reading in the UserDefined air terminal before the AirDistributionUnit object is read in. Also of note is that the UserDefined air terminal does not use the ADUNum struct variable as far as I can tell.

** Severe  ** GetUserDefinedComponents: No matching Air Distribution Unit for AirTerminal:SingleDuct:UserDefined = AIRMIXER
**   ~~~   ** ...should have outlet node=TUNNELINLET

https://unmethours.com/question/98954/error-in-userdefined-hvac-objects-assembled/?answer=99039#post-id-99039

UserDefinedNodeError

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version)
  • Version of EnergyPlus (if using an intermediate build, include SHA)
  • Unmethours link or helpdesk ticket number

Checklist

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

  • Defect file added (test1.zip)
  • 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)
@rraustad rraustad added the Defect Includes code to repair a defect in EnergyPlus label May 16, 2024
@mjwitte
Copy link
Contributor

mjwitte commented May 16, 2024

@rraustad There are some minor changes to UserDefined terminal units in #10372 to support the HVAC topology reports. Not sure if it has any impact on this issue.

@rraustad
Copy link
Contributor Author

@mjwitte I ran this defect file in that branch and this defect file crashes here just after where the ADUNum fails to match the outlet nodes as described in the description (same issue - AirDistUnit is not allocated). Line 2112 of UserDefinedComponents:

state.dataDefineEquipment->AirDistUnit(state.dataUserDefinedComponents->UserAirTerminal(CompLoop).ADUNum)
    .TermUnitSizingNum = state.dataZoneEquip->ZoneEquipConfig(CtrlZone).AirDistUnitCool(SupAirIn).TermUnitSizingIndex;

@mjwitte
Copy link
Contributor

mjwitte commented May 16, 2024

So, how does EMSUserDefined5ZoneAirCooled.idf run without error?

@rraustad
Copy link
Contributor Author

EMSUserDefined5ZoneAirCooled gets the ADU inputs before UserDefined getInput. This defect file seems to take a different path.

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
Development

No branches or pull requests

2 participants