Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/global_Ss' into global_utility_r…
Browse files Browse the repository at this point in the history
…outines
  • Loading branch information
brianlball committed Oct 20, 2020
2 parents 155981b + d742d68 commit 26bd975
Show file tree
Hide file tree
Showing 31 changed files with 2,020 additions and 2,150 deletions.
96 changes: 48 additions & 48 deletions src/EnergyPlus/ConvectionCoefficients.cc

Large diffs are not rendered by default.

8 changes: 4 additions & 4 deletions src/EnergyPlus/ConvectionCoefficients.hh
Original file line number Diff line number Diff line change
Expand Up @@ -458,14 +458,14 @@ namespace ConvectionCoefficients {

Real64 CalcASHRAESimpleIntConvCoeff(Real64 Tsurf, Real64 Tamb, Real64 cosTilt);

void CalcASHRAESimpleIntConvCoeff(int SurfNum, // surface number for which coefficients are being calculated
void CalcASHRAESimpleIntConvCoeff(EnergyPlusData &state, int SurfNum, // surface number for which coefficients are being calculated
Real64 SurfaceTemperature, // Temperature of surface for evaluation of HcIn
Real64 ZoneMeanAirTemperature // Mean Air Temperature of Zone
);

Real64 CalcASHRAETARPNatural(Real64 Tsurf, Real64 Tamb, Real64 cosTilt);

void CalcASHRAEDetailedIntConvCoeff(int SurfNum, // surface number for which coefficients are being calculated
void CalcASHRAEDetailedIntConvCoeff(EnergyPlusData &state, int SurfNum, // surface number for which coefficients are being calculated
Real64 SurfaceTemperature, // Temperature of surface for evaluation of HcIn
Real64 ZoneMeanAirTemperature // Mean Air Temperature of Zone
);
Expand Down Expand Up @@ -504,7 +504,7 @@ namespace ConvectionCoefficients {
Array1D<Real64> &SurfaceTemperatures // For CalcASHRAEDetailed, if called
);

void CalcTrombeWallIntConvCoeff(int ZoneNum, // Zone number for which coefficients are being calculated
void CalcTrombeWallIntConvCoeff(EnergyPlusData &state, int ZoneNum, // Zone number for which coefficients are being calculated
const Array1D<Real64> &SurfaceTemperatures // Temperature of surfaces for evaluation of HcIn
);

Expand Down Expand Up @@ -553,7 +553,7 @@ namespace ConvectionCoefficients {

void EvaluateExtHcModels(EnergyPlusData &state, int SurfNum, int NaturalConvModelEqNum, int ForcedConvModelEqNum, Real64 &Hc);

void DynamicExtConvSurfaceClassification(int SurfNum); // surface number
void DynamicExtConvSurfaceClassification(EnergyPlusData &state, int SurfNum); // surface number

void MapExtConvClassificationToHcModels(EnergyPlusData &state, int SurfNum); // surface number

Expand Down
1 change: 1 addition & 0 deletions src/EnergyPlus/Data/CommonIncludes.hh
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@
#include <EnergyPlus/SplitterComponent.hh>
#include <EnergyPlus/SteamBaseboardRadiator.hh>
#include <EnergyPlus/SteamCoils.hh>
#include <EnergyPlus/SurfaceGeometry.hh>
#include <EnergyPlus/SurfaceGroundHeatExchanger.hh>
#include <EnergyPlus/SwimmingPool.hh>
#include <EnergyPlus/SystemAvailabilityManager.hh>
Expand Down
2 changes: 2 additions & 0 deletions src/EnergyPlus/Data/EnergyPlusData.cc
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,7 @@ namespace EnergyPlus {
this->dataSplitterComponent = std::unique_ptr<SplitterComponentData>(new SplitterComponentData);
this->dataSteamBaseboardRadiator = std::unique_ptr<SteamBaseboardRadiatorData>(new SteamBaseboardRadiatorData);
this->dataSteamCoils = std::unique_ptr<SteamCoilsData>(new SteamCoilsData);
this->dataSurfaceGeometry = std::unique_ptr<SurfaceGeometryData>(new SurfaceGeometryData);
this->dataSurfaceGroundHeatExchangers = std::unique_ptr<SurfaceGroundHeatExchangersData>(new SurfaceGroundHeatExchangersData);
this->dataSwimmingPools = std::unique_ptr<SwimmingPoolsData>(new SwimmingPoolsData);
this->dataSystemAvailabilityManager = std::unique_ptr<SystemAvailabilityManagerData>(new SystemAvailabilityManagerData);
Expand Down Expand Up @@ -158,6 +159,7 @@ namespace EnergyPlus {
this->dataSplitterComponent->clear_state();
this->dataSteamBaseboardRadiator->clear_state();
this->dataSteamCoils->clear_state();
this->dataSurfaceGeometry->clear_state();
this->dataSurfaceGroundHeatExchangers->clear_state();
this->dataSwimmingPools->clear_state();
this->dataSystemAvailabilityManager->clear_state();
Expand Down
2 changes: 2 additions & 0 deletions src/EnergyPlus/Data/EnergyPlusData.hh
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,7 @@ struct SolarShadingData;
struct SplitterComponentData;
struct SteamBaseboardRadiatorData;
struct SteamCoilsData;
struct SurfaceGeometryData;
struct SurfaceGroundHeatExchangersData;
struct SwimmingPoolsData;
struct SystemAvailabilityManagerData;
Expand Down Expand Up @@ -171,6 +172,7 @@ struct EnergyPlusData : BaseGlobalStruct {
std::unique_ptr<SplitterComponentData> dataSplitterComponent;
std::unique_ptr<SteamBaseboardRadiatorData> dataSteamBaseboardRadiator;
std::unique_ptr<SteamCoilsData> dataSteamCoils;
std::unique_ptr<SurfaceGeometryData> dataSurfaceGeometry;
std::unique_ptr<SurfaceGroundHeatExchangersData> dataSurfaceGroundHeatExchangers;
std::unique_ptr<SwimmingPoolsData> dataSwimmingPools;
std::unique_ptr<SystemAvailabilityManagerData> dataSystemAvailabilityManager;
Expand Down
10 changes: 5 additions & 5 deletions src/EnergyPlus/HeatBalanceKivaManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -681,7 +681,7 @@ namespace HeatBalanceKivaManager {
bool useDetailedExposedPerimeter = false;
Real64 exposedFraction = 0.0;

auto &expPerimMap = SurfaceGeometry::exposedFoundationPerimeter.surfaceMap;
auto &expPerimMap = state.dataSurfaceGeometry->exposedFoundationPerimeter.surfaceMap;
if (expPerimMap.count(surfNum) == 1) {
userSetExposedPerimeter = true;
useDetailedExposedPerimeter = expPerimMap[surfNum].useDetailedExposedPerimeter;
Expand Down Expand Up @@ -1091,7 +1091,7 @@ namespace HeatBalanceKivaManager {
// Start with steady-state solution
kv.initGround(state, kivaWeather);
}
calcKivaSurfaceResults();
calcKivaSurfaceResults(state);
}

void KivaManager::calcKivaInstances(EnergyPlusData &state)
Expand All @@ -1106,7 +1106,7 @@ namespace HeatBalanceKivaManager {
}
}

calcKivaSurfaceResults();
calcKivaSurfaceResults(state);
}

void KivaInstanceMap::plotDomain()
Expand Down Expand Up @@ -1192,14 +1192,14 @@ namespace HeatBalanceKivaManager {
#endif
}

void KivaManager::calcKivaSurfaceResults()
void KivaManager::calcKivaSurfaceResults(EnergyPlusData &state)
{
for (int surfNum = 1; surfNum <= (int)DataSurfaces::Surface.size(); ++surfNum) {
if (DataSurfaces::Surface(surfNum).ExtBoundCond == DataSurfaces::KivaFoundation) {
std::string contextStr = "Surface=\"" + DataSurfaces::Surface(surfNum).Name + "\"";
Kiva::setMessageCallback(kivaErrorCallback, &contextStr);
surfaceMap[surfNum].calc_weighted_results();
DataHeatBalance::HConvIn(surfNum) = SurfaceGeometry::kivaManager.surfaceMap[surfNum].results.hconv;
DataHeatBalance::HConvIn(surfNum) = state.dataSurfaceGeometry->kivaManager.surfaceMap[surfNum].results.hconv;
}
}
Kiva::setMessageCallback(kivaErrorCallback, nullptr);
Expand Down
2 changes: 1 addition & 1 deletion src/EnergyPlus/HeatBalanceKivaManager.hh
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ namespace HeatBalanceKivaManager {
void defineDefaultFoundation();
void addDefaultFoundation();
int findFoundation(std::string const &name);
void calcKivaSurfaceResults();
void calcKivaSurfaceResults(EnergyPlusData &state);

KivaWeatherData kivaWeather;
FoundationKiva defaultFoundation;
Expand Down
22 changes: 11 additions & 11 deletions src/EnergyPlus/HeatBalanceSurfaceManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2268,9 +2268,9 @@ namespace HeatBalanceSurfaceManager {

// Initialize Kiva convection algorithms
if (Surface(SurfNum).ExtBoundCond == DataSurfaces::KivaFoundation) {
SurfaceGeometry::kivaManager.surfaceConvMap[SurfNum].in = KIVA_CONST_CONV(3.076);
SurfaceGeometry::kivaManager.surfaceConvMap[SurfNum].f = KIVA_HF_DEF;
SurfaceGeometry::kivaManager.surfaceConvMap[SurfNum].out = KIVA_CONST_CONV(0.0);
state.dataSurfaceGeometry->kivaManager.surfaceConvMap[SurfNum].in = KIVA_CONST_CONV(3.076);
state.dataSurfaceGeometry->kivaManager.surfaceConvMap[SurfNum].f = KIVA_HF_DEF;
state.dataSurfaceGeometry->kivaManager.surfaceConvMap[SurfNum].out = KIVA_CONST_CONV(0.0);
}

// Initialize the flux histories
Expand Down Expand Up @@ -6311,7 +6311,7 @@ namespace HeatBalanceSurfaceManager {

// Initialize Kiva instances ground temperatures
if (DataHeatBalance::AnyKiva) {
SurfaceGeometry::kivaManager.initKivaInstances(state);
state.dataSurfaceGeometry->kivaManager.initKivaInstances(state);
}
}
if (!state.dataGlobal->BeginEnvrnFlag) {
Expand Down Expand Up @@ -6508,10 +6508,10 @@ namespace HeatBalanceSurfaceManager {

// Calculate Kiva instances
if (DataHeatBalance::AnyKiva) {
if (((SurfaceGeometry::kivaManager.settings.timestepType == HeatBalanceKivaManager::KivaManager::Settings::HOURLY && TimeStep == 1) ||
SurfaceGeometry::kivaManager.settings.timestepType == HeatBalanceKivaManager::KivaManager::Settings::TIMESTEP) &&
if (((state.dataSurfaceGeometry->kivaManager.settings.timestepType == HeatBalanceKivaManager::KivaManager::Settings::HOURLY && TimeStep == 1) ||
state.dataSurfaceGeometry->kivaManager.settings.timestepType == HeatBalanceKivaManager::KivaManager::Settings::TIMESTEP) &&
!WarmupFlag) {
SurfaceGeometry::kivaManager.calcKivaInstances(state);
state.dataSurfaceGeometry->kivaManager.calcKivaInstances(state);
}
}

Expand All @@ -6521,7 +6521,7 @@ namespace HeatBalanceSurfaceManager {
TempInsOld = TempSurfIn; // Keep track of last iteration's temperature values

if (DataHeatBalance::AnyKiva) {
for (auto &kivaSurf : SurfaceGeometry::kivaManager.surfaceMap) {
for (auto &kivaSurf : state.dataSurfaceGeometry->kivaManager.surfaceMap) {
TempSurfIn(kivaSurf.first) = kivaSurf.second.results.Tavg - DataGlobalConstants::KelvinConv(); // TODO: Use average radiant temp? Trad?
}
}
Expand All @@ -6530,7 +6530,7 @@ namespace HeatBalanceSurfaceManager {
state, TempSurfIn, InsideSurfIterations, SurfNetLWRadToSurf, ZoneToResimulate, Inside); // Update the radiation balance

if (DataHeatBalance::AnyKiva) {
for (auto &kivaSurf : SurfaceGeometry::kivaManager.surfaceMap) {
for (auto &kivaSurf : state.dataSurfaceGeometry->kivaManager.surfaceMap) {
TempSurfIn(kivaSurf.first) = TempInsOld(kivaSurf.first);
}
}
Expand Down Expand Up @@ -6813,8 +6813,8 @@ namespace HeatBalanceSurfaceManager {

} else if (surface.HeatTransferAlgorithm == HeatTransferModel_Kiva) {
// Read Kiva results for each surface
TempSurfInTmp(SurfNum) = SurfaceGeometry::kivaManager.surfaceMap[SurfNum].results.Tconv - DataGlobalConstants::KelvinConv();
SurfOpaqInsFaceConductionFlux(SurfNum) = SurfaceGeometry::kivaManager.surfaceMap[SurfNum].results.qtot;
TempSurfInTmp(SurfNum) = state.dataSurfaceGeometry->kivaManager.surfaceMap[SurfNum].results.Tconv - DataGlobalConstants::KelvinConv();
SurfOpaqInsFaceConductionFlux(SurfNum) = state.dataSurfaceGeometry->kivaManager.surfaceMap[SurfNum].results.qtot;
SurfOpaqInsFaceConduction(SurfNum) = SurfOpaqInsFaceConductionFlux(SurfNum) * DataSurfaces::Surface(SurfNum).Area;

TH11 = 0.0;
Expand Down
2 changes: 1 addition & 1 deletion src/EnergyPlus/SimulationManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -605,7 +605,7 @@ namespace SimulationManager {

if (DataEnvironment::varyingLocationSchedIndexLat > 0 || DataEnvironment::varyingLocationSchedIndexLong > 0 ||
DataEnvironment::varyingOrientationSchedIndex > 0) {
WeatherManager::UpdateLocationAndOrientation();
WeatherManager::UpdateLocationAndOrientation(state);
}

state.dataGlobal->BeginTimeStepFlag = true;
Expand Down
4 changes: 4 additions & 0 deletions src/EnergyPlus/StateManagement.cc
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,10 @@
#include <EnergyPlus/SingleDuct.hh>
#include <EnergyPlus/SizingAnalysisObjects.hh>
#include <EnergyPlus/SizingManager.hh>
#include <EnergyPlus/SolarCollectors.hh>
#include <EnergyPlus/SolarShading.hh>
#include <EnergyPlus/SplitterComponent.hh>
#include <EnergyPlus/SteamCoils.hh>
#include <EnergyPlus/SurfaceGeometry.hh>

void EnergyPlus::clearAllStates(EnergyPlusData &state)
Expand Down
Loading

4 comments on commit 26bd975

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

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

global_utility_routines (brianlball) - x86_64-MacOS-10.15-clang-11.0.0: Build Failed

Failures:\n

API Test Summary

  • Failed: 6
  • notrun: 3

integration Test Summary

  • Passed: 2
  • Failed: 717

regression Test Summary

  • Passed: 715
  • Failed: 2

Build Badge Test Badge

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

global_utility_routines (brianlball) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: Build Failed

Failures:\n

API Test Summary

  • Failed: 6
  • notrun: 3

integration Test Summary

  • Passed: 2
  • Failed: 720

regression Test Summary

  • Passed: 735
  • Failed: 2

Build Badge Test Badge

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

global_utility_routines (brianlball) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: Build Failed

Failures:\n

API Test Summary

  • Failed: 6
  • notrun: 3

Build Badge Test Badge Coverage Badge

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

global_utility_routines (brianlball) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: Build Failed

Failures:\n

integration Test Summary

  • Passed: 2
  • Failed: 720

Build Badge Test Badge Coverage Badge

Please sign in to comment.