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
Addressed EIRFuelFiredHeatPump object electric energy consumption output variable reports zero value #10425
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code change walk-through.
@@ -1862,8 +1862,7 @@ void EIRFuelFiredHeatPump::doPhysics(EnergyPlusData &state, Real64 currentLoad) | |||
auto &thisInletNode = state.dataLoopNodes->Node(this->loadSideNodes.inlet); | |||
auto &thisOutletNode = state.dataLoopNodes->Node(this->loadSideNodes.outlet); | |||
auto &sim_component = DataPlant::CompData::getPlantComponent(state, this->loadSidePlantLoc); | |||
bool RunFlag = true; | |||
if ((this->EIRHPType == DataPlant::PlantEquipmentType::HeatPumpFuelFiredHeating && currentLoad <= 0.0) || !RunFlag) { | |||
if (this->EIRHPType == DataPlant::PlantEquipmentType::HeatPumpFuelFiredHeating && currentLoad <= 0.0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The RunFlag
flag is always true,
so it is redundant logic and must be removed.
@@ -2250,7 +2249,7 @@ void EIRFuelFiredHeatPump::doPhysics(EnergyPlusData &state, Real64 currentLoad) | |||
this->powerUsage += this->standbyElecPower; | |||
|
|||
this->fuelEnergy = this->fuelRate * reportingInterval; | |||
this->powerEnergy = this->powerEnergy * reportingInterval; | |||
this->powerEnergy = this->powerUsage * reportingInterval; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using the correct equation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, good fix.
@@ -4363,6 +4364,7 @@ TEST_F(EnergyPlusFixture, GAHP_HeatingSimulate_AirSource_with_Defrost) | |||
// expect it to meet setpoint and have some pre-evaluated conditions | |||
// EXPECT_NEAR(specifiedLoadSetpoint, thisHeatingPLHP->loadSideOutletTemp, 0.001); | |||
EXPECT_NEAR(curLoad, thisHeatingPLHP->loadSideHeatTransfer, 0.001); | |||
EXPECT_NEAR(18020.0, thisEIRPlantLoopHP->powerUsage, 0.001); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modified an existing unit test to show the fix.
@@ -4381,6 +4383,7 @@ TEST_F(EnergyPlusFixture, GAHP_HeatingSimulate_AirSource_with_Defrost) | |||
thisHeatingPLHP->simulate(*state, myLoadLocation, firstHVAC, curLoad, runFlag); | |||
EXPECT_NEAR(28800.0, thisHeatingPLHP->fuelRate, 0.001); | |||
EXPECT_NEAR(25920000.0, thisHeatingPLHP->fuelEnergy, 0.001); | |||
EXPECT_NEAR(18020.0, thisEIRPlantLoopHP->powerUsage, 0.001); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modified an existing unit test to show the fix.
@@ -4402,6 +4405,7 @@ TEST_F(EnergyPlusFixture, GAHP_HeatingSimulate_AirSource_with_Defrost) | |||
// expect it to miss setpoint and be at max capacity | |||
EXPECT_NEAR(45.0, thisHeatingPLHP->loadSideOutletTemp, 0.001); | |||
EXPECT_NEAR(30.0, thisHeatingPLHP->sourceSideOutletTemp, 0.001); | |||
EXPECT_NEAR(0.0, thisEIRPlantLoopHP->powerUsage, 0.001); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above.
The difference in regression results is expected in an example idf |
I see the |
One file with big diffs.....as you would expect when we were calculating the energy report variable incorrectly. Good to go. Thanks @Nigusse ! |
Pull request overview
Fuel-fired Absorption HeatPump Electricity Energy
report variable, which was always zero prior to this fix.NOTE: ENHANCEMENTS MUST FOLLOW A SUBMISSION PROCESS INCLUDING A FEATURE PROPOSAL AND DESIGN DOCUMENT PRIOR TO SUBMITTING CODE
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.