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

Crash in Fluid Cooler Code #10817

Open
Myoldmopar opened this issue Nov 8, 2024 · 2 comments · May be fixed by #10855
Open

Crash in Fluid Cooler Code #10817

Myoldmopar opened this issue Nov 8, 2024 · 2 comments · May be fixed by #10855
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@Myoldmopar
Copy link
Member

Issue overview

The fluid cooler sizing routine assumes the user has provided a sizing:plant object without checking. This leads to a hard crash in the file linked below.

Checklist

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

@Myoldmopar Myoldmopar added the Defect Includes code to repair a defect in EnergyPlus label Nov 8, 2024
@jmarrec jmarrec self-assigned this Dec 13, 2024
jmarrec added a commit that referenced this issue Dec 13, 2024
jmarrec added a commit that referenced this issue Dec 13, 2024
…ater flow rate autosized and no Sizing:Plant
@jmarrec
Copy link
Contributor

jmarrec commented Dec 13, 2024

EvaporativeFluidCooler has the same issue.

@jmarrec
Copy link
Contributor

jmarrec commented Dec 13, 2024

Other problems found in FluidCoolers.cc:

DesignLeavingWaterTemp not set in almost all cases

DesignLeavingWaterTemp is ONLY ever assigned when DesignWaterFlowRateWasAutoSized here

this->DesignLeavingWaterTemp = state.dataSize->PlantSizData(PltSizCondNum).ExitTemp;

Yet it is used to report itself, the approach, and the range to the tabular output here

OutputReportPredefined::PreDefTableEntry(
state, state.dataOutRptPredefined->pdchCTFCRange, this->Name, this->DesignEnteringWaterTemp - this->DesignLeavingWaterTemp);
OutputReportPredefined::PreDefTableEntry(
state, state.dataOutRptPredefined->pdchCTFCApproach, this->Name, this->DesignLeavingWaterTemp - this->DesignEnteringAirWetBulbTemp);
OutputReportPredefined::PreDefTableEntry(
state, state.dataOutRptPredefined->pdchCTFCDesFanPwr, this->Name, this->HighSpeedFanPower); // equivalent to Design Fan Power?
OutputReportPredefined::PreDefTableEntry(
state, state.dataOutRptPredefined->pdchCTFCDesInletAirWBT, this->Name, this->DesignEnteringAirWetBulbTemp);
OutputReportPredefined::PreDefTableEntry(state, state.dataOutRptPredefined->pdchCTFCDesWaterFlowRate, this->Name, this->DesignWaterFlowRate);
OutputReportPredefined::PreDefTableEntry(state, state.dataOutRptPredefined->pdchCTFCLevWaterSPTemp, this->Name, this->DesignLeavingWaterTemp);

Precision for reporting DesignWaterFlowRate

It is defaulted to 2, which is insufficient

OutputReportPredefined::PreDefTableEntry(state, state.dataOutRptPredefined->pdchCTFCDesWaterFlowRate, this->Name, this->DesignWaterFlowRate);


Example file tabular output

See the testfiles/FluidCooler.idf: energyplus -w $chig_epw -d out FluidCooler.idf

image

Same in markdown, removing some columns for clarity

Range [C] Approach [C] Design Fan Power [W] Design Inlet Air Wet-Bulb Temperature [C] Design Water Flow Rate [m3/s] Leaving Water Setpoint Temperature [C]
BIG FLUIDCOOLER 51.67 -25.60 615.31 25.60 0.00 0.00

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
2 participants