Skip to content

Adjust low flow warning for w2a coil in cycling fan mode #11485

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.

Sign up for GitHub

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

Open
joseph-robertson wants to merge 14 commits into
base: develop
Choose a base branch
from
+85 −32
Open

Adjust low flow warning for w2a coil in cycling fan mode #11485

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
ed5a7fa
Adjust low flow warning for cycling fan.
joseph-robertson Mar 27, 2026
e0be50a
Formatting.
joseph-robertson Mar 27, 2026
dcb96e7
Pass PartLoadRation into InitSimpleWatertoAirHP.
joseph-robertson Mar 27, 2026
cca85cb
Update unit tests for update to InitSimpleWatertoAirHP.
joseph-robertson Mar 27, 2026
9ec1457
Formatting.
joseph-robertson Mar 27, 2026
0a5be8c
Merge branch 'develop' into coil-w2a-cycling-fan
joseph-robertson Apr 8, 2026
e8f0d32
Merge branch 'develop' into coil-w2a-cycling-fan
mitchute Apr 9, 2026
1ced44d
Move low flow flag to conditions struct so each coil can report the w…
joseph-robertson Apr 9, 2026
dfd83ab
Coil in cycling fan mode with PLR=0 should throw the low flow warning.
joseph-robertson Apr 9, 2026
db02101
Update unit test to check for recurring warning with continuous fan.
joseph-robertson Apr 9, 2026
8639482
Coil is off when PLR=0 for cycling fan.
joseph-robertson Apr 13, 2026
0590663
Add unit test for cycling fan low flow.
joseph-robertson Apr 13, 2026
550577d
Merge branch 'develop' into coil-w2a-cycling-fan
joseph-robertson Apr 13, 2026
ad69260
Formatting.
joseph-robertson Apr 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 19 additions & 7 deletions src/EnergyPlus/WaterToAirHeatPumpSimple.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -911,7 +911,7 @@ namespace WaterToAirHeatPumpSimple {
int const HPNum, // Current HPNum under simulation
Real64 const SensLoad, // Control zone sensible load[W]
Real64 const LatentLoad, // Control zone latent load[W]
[[maybe_unused]] HVAC::FanOp const fanOp, // fan operating mode
HVAC::FanOp const fanOp, // fan operating mode
[[maybe_unused]] Real64 const OnOffAirFlowRatio, // ratio of compressor on flow to average flow over time step
bool const FirstHVACIteration // Iteration flag
)
Expand Down Expand Up @@ -1107,12 +1107,24 @@ namespace WaterToAirHeatPumpSimple {
state.dataLoopNodes->Node(AirInletNode).Temp,
state.dataLoopNodes->Node(AirInletNode).HumRat,
RoutineName);
if (simpleWatertoAirHP.AirMassFlowRate < 0.25 * RatedAirMassFlowRate) {
ShowRecurringWarningErrorAtEnd(state,
"Actual air mass flow rate is smaller than 25% of water-to-air heat pump coil rated air flow rate.",
state.dataWaterToAirHeatPumpSimple->AirflowErrPointer,
simpleWatertoAirHP.AirMassFlowRate,
simpleWatertoAirHP.AirMassFlowRate);
if (fanOp != HVAC::FanOp::Cycling) {
if (simpleWatertoAirHP.AirMassFlowRate < 0.25 * RatedAirMassFlowRate) {
ShowRecurringWarningErrorAtEnd(state,
"Actual air mass flow rate is smaller than 25% of water-to-air heat pump coil rated air flow rate.",
state.dataWaterToAirHeatPumpSimple->AirflowErrPointer,
simpleWatertoAirHP.AirMassFlowRate,
simpleWatertoAirHP.AirMassFlowRate);
}
} else {
if ((simpleWatertoAirHP.AirMassFlowRate / simpleWatertoAirHP.PartLoadRatio) < 0.25 * RatedAirMassFlowRate) {
if (state.dataWaterToAirHeatPumpSimple->LowFlowFlag) {
ShowWarningError(state,
Copy link
Copy Markdown
Collaborator Author

@joseph-robertson joseph-robertson Mar 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way we could alternatively issue this warning once, upstream, using parent airflow rate and rated coil airflow rate? See #8980 (comment).

Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think leaving this check in the coil object is better than creating code in 3 parent objects. You could just add a simpleWatertoAirHP.OneTimeFlag around these checks and that should be sufficient. The parent could do this, after sizing, and comparing with coil design flow, but the same 1-time technique would still be used.

EnergyPlus::format("{}: Actual air mass flow rate is smaller than 25% of water-to-air heat pump coil ({}) rated air flow rate.",
RoutineName,
simpleWatertoAirHP.Name));
state.dataWaterToAirHeatPumpSimple->LowFlowFlag = false;
}
}
Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same thing that is done in the calc function so this is correct.

void CalcHPCooling(Heating)Simple:

    if (fanOp == HVAC::FanOp::Continuous) {
        LoadSideFullMassFlowRate = simpleWatertoAirHP.AirMassFlowRate;
    } else {
        // default to cycling fan, cycling compressor, full load air flow
        if (PartLoadRatio > 0.0) {
            LoadSideFullMassFlowRate = simpleWatertoAirHP.AirMassFlowRate / PartLoadRatio;
        } else {
            LoadSideFullMassFlowRate = 0.0;
        }
    }

Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joseph-robertson I see the recurring removed from the err file, but where is the 1-time warning? In this diff, shouldn't there be a new warning that wasn't there before ? ASHRAE901_ApartmentHighRise_STD2019_Denver

Image

Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 8, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, this should be a SimpleWatertoAirHPConditions struct variable so each coil can report the warning.

if (state.dataWaterToAirHeatPumpSimple->LowFlowFlag) {

Copy link
Copy Markdown
Collaborator Author

@joseph-robertson joseph-robertson Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rraustad I think previously the recurring warning was being thrown on this testfile for coils in cycling fan mode. With the new / PartLoadRatio, the warning goes away. It's possible there are no testfiles (with coils in continuous fan mode) where we should expect the 1-time warning to be thrown. Perhaps I should look into writing a couple new unit tests for showing that both warning types get thrown under the right conditions; one for coil in continuous fan mode and one for coil in cycling fan mode.

Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duh, your right.

Copy link
Copy Markdown
Collaborator Author

@joseph-robertson joseph-robertson Apr 9, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the divide by zero fix in dfd83ab, the testfile DOAToWaterToAirHPInlet_MultispeedFan.idf does throw the 1-time warning.

image

Copy link
Copy Markdown
Collaborator

@rraustad rraustad Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the node air flow rate when that warning is hit? If it's 0 then that warning should not be reported. Even if the flow rate is > 0 the coil will be off. Why report a warning for a time step when the coil is off? Eventually the coil will be on and that check will throw if necessary. For cycling fan if PLR = 0 then the coil will be off and the fan will be off. A parent can still call the coil with PLR = 0 when there is actually a load and the result will be capacity = 0. A parent will call the coil model with PLR = 0 and PLR = 1 to test whether either of these can meet the load. For continuous fan and PLR = 0 there will be a non-zero capacity because of fan heat and also if OA is provided and that result could meet the load so the parent will not turn on the coil. The check should be testing for if the coil is on, which also protects the divide by 0 but that's a different thing.

if (PartLoadRatio > 0.0 && (simpleWAHP.AirMassFlowRate / PartLoadRatio) < 0.25 * RatedAirMassFlowRate)

Copy link
Copy Markdown
Collaborator Author

@joseph-robertson joseph-robertson Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will make this change. (I do see that the node air flow rate > 0 when PLR = 0.)

Since we'll now have no testfiles throwing this warning, I'll see if I can add a unit test for demonstrating it.

}
simpleWatertoAirHP.WaterFlowMode = true;
} else { // heat pump is off
Expand Down
1 change: 1 addition & 0 deletions src/EnergyPlus/WaterToAirHeatPumpSimple.hh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,7 @@ struct WaterToAirHeatPumpSimpleData : BaseGlobalStruct
Real64 QLatActual; // Actual Latent Capacity [W]
Real64 Winput; // Power Consumption [W]
bool MyOneTimeFlag = true; // one time allocation flag
bool LowFlowFlag = true;

Array1D<WaterToAirHeatPumpSimple::SimpleWatertoAirHPConditions> SimpleWatertoAirHP;

Expand Down
Loading