Skip to content

Commit

Permalink
more Plant sizing index corrections.
Browse files Browse the repository at this point in the history
fixed more places where plant loop index was used instead of plant
sizing index.
  • Loading branch information
EnergyArchmage committed Mar 13, 2015
1 parent 0580201 commit fcdc490
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/EnergyPlus/SizingAnalysisObjects.cc
Original file line number Diff line number Diff line change
Expand Up @@ -519,13 +519,13 @@ namespace EnergyPlus {

// now apply the correct sizing factor depending on input option
sizingFac = 1.0;
if ( PlantSizData( plantLoopIndex ).SizingFactorOption == NoSizingFactorMode ) {
if ( PlantSizData( plantSizingIndex ).SizingFactorOption == NoSizingFactorMode ) {
sizingFac = 1.0;
} else if ( PlantSizData( plantLoopIndex ).SizingFactorOption == GlobalHeatingSizingFactorMode ) {
} else if ( PlantSizData( plantSizingIndex ).SizingFactorOption == GlobalHeatingSizingFactorMode ) {
sizingFac = GlobalHeatSizingFactor;
} else if ( PlantSizData( plantLoopIndex ).SizingFactorOption == GlobalCoolingSizingFactorMode ) {
} else if ( PlantSizData( plantSizingIndex ).SizingFactorOption == GlobalCoolingSizingFactorMode ) {
sizingFac = GlobalCoolSizingFactor;
} else if ( PlantSizData( plantLoopIndex ).SizingFactorOption == LoopComponentSizingFactorMode ) {
} else if ( PlantSizData( plantSizingIndex ).SizingFactorOption == LoopComponentSizingFactorMode ) {
//multiplier used for pumps, often 1.0, from component level sizing fractions
sizingFac = PlantLoop( plantLoopIndex ).LoopSide( SupplySide ).Branch( 1 ).PumpSizFac;
}
Expand All @@ -551,7 +551,7 @@ namespace EnergyPlus {

if ( setNewSizes ) {
// set new size values for rest of simulation
PlantSizData( plantLoopIndex ).DesVolFlowRate = newVolDesignFlowRate;
PlantSizData( plantSizingIndex ).DesVolFlowRate = newVolDesignFlowRate;

if (PlantLoop( plantLoopIndex ).MaxVolFlowRateWasAutoSized ) {
PlantLoop( plantLoopIndex ).MaxVolFlowRate = newVolDesignFlowRate;
Expand Down

7 comments on commit fcdc490

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-MacOS-10.9-clang: Tests Failed

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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

Build 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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 647 tests passed)

Build Badge Test Badge Coverage 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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: Tests Failed

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.

HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1155 of 1156 tests passed)

Build Badge Test Badge Coverage Badge

Please sign in to comment.