NREL / EnergyPlus

EnergyPlus™ is a whole building energy simulation program that engineers, architects, and researchers use to model both energy consumption and water use in buildings.
https://energyplus.net
Other
1.14k stars 392 forks source link

Water to Water Heat Pump output variable name conflict #7733

Closed mjwitte closed 4 years ago

mjwitte commented 4 years ago

Issue overview

In v9.2, all of the HeatPump:WaterToWater:* objects had output variables which began with "Water to Water Heat Pump . . . ".

In #7489, HeatPump:WaterToWater:EIR:Cooling became HeatPump:PlantLoop:EIR:Cooling and was extended to include an air-source option, so it's output variables were all changed to "~Water to Water~ Heat Pump . . . ". The transition Report Variables 9-2-0 to 9-3-0.csv implements this change.

But HeatPump:WaterToWater:ParameterEstimation: and HeatPump:WaterToWater:EquationFit: still use the old output variable names, but transition doesn't know the difference so it will change the names and break idf files which have these heat pumps in them.

The simplest solution is probably to update the names for HeatPump:WaterToWater:ParameterEstimation: and HeatPump:WaterToWater:EquationFit: to drop "Water to Water . . .".

Details

Some additional details for this issue (if relevant):

Checklist

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

mitchute commented 4 years ago

"... HeatPump:WaterToWater:ParameterEstimation: and HeatPump:WaterToWater:EquationFit: still use the old output variable names..."

Of course they do... I'll submit another PR to fix this up.

mitchute commented 4 years ago

Closed by #7734