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

Inconsistent maximum flow rate for fans and terminals when sizing on ventilation needs #10438

Open
2 tasks
lymereJ opened this issue Mar 14, 2024 · 0 comments
Open
2 tasks
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@lymereJ
Copy link
Collaborator

lymereJ commented Mar 14, 2024

Issue overview

While working on #10368 I noticed that in the PlantLoopHeatPump_EIR_Large-Office-2-AWHP-DedHR-AuxBoiler-Pri-Sec-HW.idf test file that the maximum flow rate for the fan terminal is actually sized (based on ventilation needs) to be larger than the maximum terminal flow rate. For PIUs, this was previously not much of an issue because only constant volume fan operation were allowed.

Running the file in the debugger shows that the terminal is sized first and the fan second. Both of them are sized to the largest of the design heating/cooling zone air flow, but the terminal is further adjusted, or scaled, in UpdateSysSizing based on the zone OA to maximum terminal flow rate. The fan maximum flow rate is never adjusted accordingly. Fan sizing is done based on data in FinalZoneSizing whereas the adjustment to the terminal is done in data in TermUnitFinalZoneSizing.

if ((SysHeatSizingRat != 1.0) && (finalSysSizing.loadSizingType == DataSizing::LoadSizing::Ventilation) &&
(termUnitFinalZoneSizing.MinOA > 0.0)) {
// size on ventilation load
ZoneOARatio = termUnitFinalZoneSizing.MinOA / max(termUnitFinalZoneSizing.DesHeatVolFlow, termUnitFinalZoneSizing.MinOA);
ZoneOARatio *= (1.0 + state.dataSize->TermUnitSizing(TermUnitSizingIndex).InducRat);
termUnitFinalZoneSizing.scaleZoneHeating(ZoneOARatio);

Details

Some additional details for this issue (if relevant):

  • Platform: Windows
  • Version of EnergyPlus: 24.1

Checklist

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

  • 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)
@lymereJ lymereJ added the Defect Includes code to repair a defect in EnergyPlus label Mar 14, 2024
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

1 participant