Skip to content

Commit

Permalink
Revert Warn if chiller chills when off
Browse files Browse the repository at this point in the history
  • Loading branch information
mjwitte committed Oct 18, 2024
1 parent 394a2c7 commit 172283a
Showing 1 changed file with 1 addition and 26 deletions.
27 changes: 1 addition & 26 deletions src/EnergyPlus/ChillerElectricEIR.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2585,7 +2585,7 @@ void ElectricEIRChillerSpecs::update(EnergyPlusData &state, Real64 const MyLoad,
Real64 ReportingConstant = state.dataHVACGlobal->TimeStepSysSec;

if (MyLoad >= 0 || !RunFlag) { // Chiller not running so pass inlet states to outlet states
// Set node conditions
// Set node conditions
state.dataLoopNodes->Node(this->EvapOutletNodeNum).Temp = state.dataLoopNodes->Node(this->EvapInletNodeNum).Temp;
state.dataLoopNodes->Node(this->CondOutletNodeNum).Temp = state.dataLoopNodes->Node(this->CondInletNodeNum).Temp;
if (this->CondenserType != DataPlant::CondenserType::WaterCooled) {
Expand Down Expand Up @@ -2682,31 +2682,6 @@ void ElectricEIRChillerSpecs::update(EnergyPlusData &state, Real64 const MyLoad,
this->HeatRecMassFlow = state.dataLoopNodes->Node(this->HeatRecInletNodeNum).MassFlowRate;
}
}
if (((this->EvapMassFlowRate < DataBranchAirLoopPlant::MassFlowTolerance) ||
(this->CondMassFlowRate < DataBranchAirLoopPlant::MassFlowTolerance)) &&
(this->EvapInletTemp != this->EvapOutletTemp)) {
if (!state.dataGlobal->WarmupFlag && !state.dataGlobal->DoingHVACSizingSimulations &&
state.dataPlnt->PlantLoop(this->CWPlantLoc.loopNum).LoopSide(this->CWPlantLoc.loopSideNum).FlowLock == DataPlant::FlowLock::Locked) {
ShowWarningError(state,
format("Chiller should be off, evap flow rate={}, but EvapInletTemp={} is != EvapOutletTemp={}, EvapInletNodeNum.Temp "
"={}, EvapOutletNodeNum).Temp={}",
this->EvapMassFlowRate,
this->EvapInletTemp,
this->EvapOutletTemp,
state.dataLoopNodes->Node(this->EvapInletNodeNum).Temp,
state.dataLoopNodes->Node(this->EvapOutletNodeNum).Temp));
ShowContinueErrorTimeStamp(state,
format("MyLoad={}, RunFlag={}, QEvaporator={}, QCondenser={}, EvapMassFlowRate={}, "
"CondMassFlowRate = {}, Power={} ",
MyLoad,
RunFlag,
this->QEvaporator,
this->QCondenser,
this->EvapMassFlowRate,
this->CondMassFlowRate,
this->Power));
}
}
}

bool ElectricEIRChillerSpecs::thermosiphonDisabled(EnergyPlusData &state)
Expand Down

4 comments on commit 172283a

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

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

chillerEIRrepairs (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (2916 of 2916 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

chillerEIRrepairs (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2894 of 2894 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

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

chillerEIRrepairs (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-RelWithDebInfo: Tests Failed (0 of 0 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

chillerEIRrepairs (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-RelWithDebInfo: OK (2098 of 2098 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.