Skip to content

Commit

Permalink
Merge branch 'develop' into EnhanceOutdoorAirReporting
Browse files Browse the repository at this point in the history
  • Loading branch information
JasonGlazer committed Jul 27, 2020
2 parents 4dd38f2 + 2b65de0 commit c8a5672
Show file tree
Hide file tree
Showing 441 changed files with 9,928 additions and 17,029 deletions.
6 changes: 3 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -321,9 +321,9 @@ if( BUILD_FORTRAN )
cmake_add_fortran_subdirectory(src/ReadVars PROJECT ReadVars NO_EXTERNAL_INSTALL )
set(TRANSITION_EXTRA_FLAGS "-DCMAKE_VERSION_MAJOR:STRING=${CMAKE_VERSION_MAJOR}" "-DCMAKE_VERSION_MINOR:STRING=${CMAKE_VERSION_MINOR}" "-DCMAKE_VERSION_PATCH:STRING=${CMAKE_VERSION_PATCH}")
cmake_add_fortran_subdirectory(src/Transition PROJECT Transition CMAKE_COMMAND_LINE ${TRANSITION_EXTRA_FLAGS} NO_EXTERNAL_INSTALL )
file( COPY "idd/V9-2-0-Energy+.idd" DESTINATION "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}" )
configure_file( idd/Energy+.idd.in "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/V9-3-0-Energy+.idd" )
file( COPY "src/Transition/SupportFiles/Report Variables 9-2-0 to 9-3-0.csv" DESTINATION "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}" )
file( COPY "idd/V9-3-0-Energy+.idd" DESTINATION "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}" )
configure_file( idd/Energy+.idd.in "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/V9-4-0-Energy+.idd" )
file( COPY "src/Transition/SupportFiles/Report Variables 9-3-0 to 9-4-0.csv" DESTINATION "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}" )
cmake_add_fortran_subdirectory(src/Basement PROJECT Basement NO_EXTERNAL_INSTALL )
cmake_add_fortran_subdirectory(src/HVAC-Diagram PROJECT HVAC-Diagram NO_EXTERNAL_INSTALL )
cmake_add_fortran_subdirectory(src/ParametricPreprocessor PROJECT ParametricPreprocessor NO_EXTERNAL_INSTALL )
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ \subsubsection{Inputs}\label{inputs-1-007}

Leaving this input field blank will bypass the \hyperref[controllermechanicalventilation]{Controller:MechanicalVentilation} object calculations and the minimum outdoor air flow rate will be based on the other inputs associated with this outdoor air controller object. Actual outdoor air flow rates may be higher than the minimum if free cooling is available and the object inputs are properly selected. Regardless, the maximum outdoor air flow rate is limited by the field Maximum Outdoor Air Flow Rate.

Note that the Minimum Outdoor Air Flow Rate times the Minimum Outdoor Air Schedule is a hard minimum that may override DCV or other advanced outdoor air controls specified in the \hyperref[controllermechanicalventilation]{Controller:MechanicalVentilation} object. The Maximum Fraction of Outdoor Air Schedule (if specified) times the current system supply air flow rate may limit the outdoor air flow rate set by the \hyperref[controllermechanicalventilation]{Controller:MechanicalVentilation}.
Note that the Minimum Outdoor Air Flow Rate times the Minimum Outdoor Air Schedule is a hard minimum that may override DCV or other advanced outdoor air controls specified in the \hyperref[controllermechanicalventilation]{Controller:MechanicalVentilation} object. A recursive warning is provided for users in the error file, when the override occurs at a specific time. The Maximum Fraction of Outdoor Air Schedule (if specified) times the current system supply air flow rate may limit the outdoor air flow rate set by the \hyperref[controllermechanicalventilation]{Controller:MechanicalVentilation}.

\paragraph{Field: Time of Day Economizer Control Schedule Name}\label{field-time-of-day-economizer-control-schedule-name}

Expand Down
3 changes: 2 additions & 1 deletion idd/Energy+.idd.in
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ SimulationControl,
\memo fields set to No, the corresponding Sizing object is ignored.
\memo Note also, if you want to do system sizing, you must also do zone sizing in the same
\memo run or an error will result.
\min-fields 5
\min-fields 7
A1, \field Do Zone Sizing Calculation
\note If Yes, Zone sizing is accomplished from corresponding Sizing:Zone objects
\note and autosize fields.
Expand Down Expand Up @@ -52323,6 +52323,7 @@ Coil:Heating:Fuel,
\key Propane
\key Diesel
\key Gasoline
\key Coal
\key FuelOilNo1
\key FuelOilNo2
\key OtherFuel1
Expand Down
22 changes: 7 additions & 15 deletions src/EnergyPlus/AirLoopHVACDOAS.cc
Original file line number Diff line number Diff line change
Expand Up @@ -447,15 +447,15 @@ namespace AirLoopHVACDOAS {
}
} else if (SELECT_CASE_var == "FAN:COMPONENTMODEL") {
thisDOAS.m_FanTypeNum = SimAirServingZones::Fan_ComponentModel;
Fans::GetFanIndex(state.fans, CompName, thisDOAS.m_FanIndex, errorsFound);
Fans::GetFanIndex(state, CompName, thisDOAS.m_FanIndex, errorsFound, ObjexxFCL::Optional_string_const());
thisDOAS.FanName = CompName;
if (CompNum == 1) {
thisDOAS.FanBlowTroughFlag = true;
}
OutsideAirSys(thisDOAS.m_OASystemNum).InletNodeNum(CompNum) =
Fans::GetFanInletNode(state.fans, SELECT_CASE_var, OutsideAirSys(thisDOAS.m_OASystemNum).ComponentName(CompNum), InletNodeErrFlag);
Fans::GetFanInletNode(state, SELECT_CASE_var, OutsideAirSys(thisDOAS.m_OASystemNum).ComponentName(CompNum), InletNodeErrFlag);
OutsideAirSys(thisDOAS.m_OASystemNum).OutletNodeNum(CompNum) =
Fans::GetFanOutletNode(state.fans, SELECT_CASE_var, OutsideAirSys(thisDOAS.m_OASystemNum).ComponentName(CompNum), OutletNodeErrFlag);
Fans::GetFanOutletNode(state, SELECT_CASE_var, OutsideAirSys(thisDOAS.m_OASystemNum).ComponentName(CompNum), OutletNodeErrFlag);
thisDOAS.m_FanInletNodeNum = OutsideAirSys(thisDOAS.m_OASystemNum).InletNodeNum(CompNum);
thisDOAS.m_FanOutletNodeNum = OutsideAirSys(thisDOAS.m_OASystemNum).OutletNodeNum(CompNum);
if (!(CompNum == 1 || CompNum == OutsideAirSys(thisDOAS.m_OASystemNum).NumComponents)) {
Expand Down Expand Up @@ -793,18 +793,10 @@ namespace AirLoopHVACDOAS {
int LoopOA;
int NodeNum;
Real64 SchAvailValue;
static Array1D_bool MyEnvrnFlag; // Used for initializations each begin environment flag
static bool MyOneTimeFlag(true); // Initialization flag
std::string RoutineName = "AirLoopDOAS::initAirLoopDOAS";
bool ErrorsFound = false;

if (MyOneTimeFlag) {
MyEnvrnFlag.allocate(state.dataAirLoopHVACDOAS.numAirLoopDOAS);
MyEnvrnFlag = true;
MyOneTimeFlag = false;
}

if (DataGlobals::BeginEnvrnFlag && MyEnvrnFlag(this->m_AirLoopDOASNum + 1)) {
if (DataGlobals::BeginEnvrnFlag && this->MyEnvrnFlag) {
Real64 rho;
DataSizing::CurSysNum = this->m_OASystemNum;
for (int CompNum = 1; CompNum <= DataAirLoop::OutsideAirSys(this->m_OASystemNum).NumComponents; ++CompNum) {
Expand Down Expand Up @@ -867,14 +859,14 @@ namespace AirLoopHVACDOAS {
}
}

MyEnvrnFlag(this->m_AirLoopDOASNum + 1) = false;
this->MyEnvrnFlag = false;
if (ErrorsFound) {
ShowFatalError("initAirLoopDOAS: Previous errors cause termination.");
}
}

if (!DataGlobals::BeginEnvrnFlag) {
MyEnvrnFlag(this->m_AirLoopDOASNum + 1) = true;
this->MyEnvrnFlag = true;
}

this->SumMassFlowRate = 0.0;
Expand Down Expand Up @@ -943,7 +935,7 @@ namespace AirLoopHVACDOAS {
}
bool errorsFound = false;
if (this->m_FanIndex > 0 && this->m_FanTypeNum == SimAirServingZones::Fan_ComponentModel) {
Fans::SetFanData(state.fans, this->m_FanIndex, errorsFound, Name, sizingMassFlow / DataEnvironment::StdRhoAir, 0);
Fans::SetFanData(state, this->m_FanIndex, errorsFound, Name, sizingMassFlow / DataEnvironment::StdRhoAir, 0);
Fans::Fan(this->m_FanIndex).MaxAirMassFlowRate = sizingMassFlow;
DataLoopNode::Node(this->m_FanInletNodeNum).MassFlowRateMaxAvail = sizingMassFlow;
DataLoopNode::Node(this->m_FanOutletNodeNum).MassFlowRateMaxAvail = sizingMassFlow;
Expand Down
3 changes: 2 additions & 1 deletion src/EnergyPlus/AirLoopHVACDOAS.hh
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,7 @@ namespace AirLoopHVACDOAS {
int CWBranchNum;
int CWCompNum;
int CWCtrlNodeNum;
bool MyEnvrnFlag;

// default constructor
AirLoopDOAS() // constructor
Expand All @@ -177,7 +178,7 @@ namespace AirLoopHVACDOAS {
m_FanIndex(-1), m_FanInletNodeNum(0), m_FanOutletNodeNum(0), m_FanTypeNum(0), m_HeatCoilNum(0), m_CoolCoilNum(0), ConveCount(0),
ConveIndex(0), m_HeatExchangerFlag(false), SizingOnceFlag(true), DXCoilFlag(false), FanBlowTroughFlag(false),
m_CompPointerAirLoopMixer(nullptr), m_CompPointerAirLoopSplitter(nullptr), HWLoopNum(0), HWLoopSide(0), HWBranchNum(0), HWCompNum(0),
HWCtrlNodeNum(0), CWLoopNum(0), CWLoopSide(0), CWBranchNum(0), CWCompNum(0), CWCtrlNodeNum(0)
HWCtrlNodeNum(0), CWLoopNum(0), CWLoopSide(0), CWBranchNum(0), CWCompNum(0), CWCtrlNodeNum(0), MyEnvrnFlag(true)

{
}
Expand Down
Loading

7 comments on commit c8a5672

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

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-MacOS-10.15-clang-11.0.0: Build Failed

Failures:\n

API Test Summary

  • Failed: 5
  • notrun: 3

integration Test Summary

  • Passed: 2
  • Failed: 709

regression Test Summary

  • Passed: 707
  • Failed: 2

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-Linux-Ubuntu-18.04-cppcheck: OK (0 of 0 tests passed, 0 test warnings)

Build Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-Linux-Ubuntu-18.04-custom_check: OK (13 of 13 tests passed, 0 test warnings)

Build Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: Build Failed

Failures:\n

API Test Summary

  • Failed: 5
  • notrun: 3

Build Badge Test Badge Coverage Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: Build Failed

Failures:\n

integration Test Summary

  • Passed: 2
  • Failed: 712

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.

EnhanceOutdoorAirReporting (JasonGlazer) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: Build Failed

Failures:\n

API Test Summary

  • Failed: 5
  • notrun: 3

integration Test Summary

  • Passed: 2
  • Failed: 712

regression Test Summary

  • Passed: 727
  • Failed: 2

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.

EnhanceOutdoorAirReporting (JasonGlazer) - Win64-Windows-10-VisualStudio-16: Build Failed

Failures:\n

API Test Summary

  • Failed: 1
  • notrun: 3

integration Test Summary

  • Passed: 2
  • Failed: 709

Build Badge Test Badge

Please sign in to comment.