Skip to content

Commit

Permalink
Merge branch 'develop' into 6050_ReturnFan_MinFlowFrac
Browse files Browse the repository at this point in the history
  • Loading branch information
mitchute committed Dec 11, 2020
2 parents 89b2d4f + de57172 commit f99ad62
Show file tree
Hide file tree
Showing 373 changed files with 10,488 additions and 14,450 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/documentation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
python-version: 3.7

- name: Set up LaTeX
run: sudo apt-get install texlive texlive-xetex texlive-science
run: sudo apt update && sudo apt install -y texlive texlive-xetex texlive-science

- name: Create Build Environment
run: cmake -E make_directory ${{runner.workspace}}/EnergyPlus/doc/build
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/mac_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:
shell: bash
run: cmake --build . --target energyplus -j 2

no_glcol_cache:
no_glycol_cache:
runs-on: macos-10.15

steps:
Expand Down
1 change: 1 addition & 0 deletions cmake/CompilerFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ ELSEIF ( CMAKE_COMPILER_IS_GNUCXX OR "${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang"
ADD_CXX_DEFINITIONS("-pipe") # Faster compiler processing
ADD_CXX_DEFINITIONS("-pedantic") # Turn on warnings about constructs/situations that may be non-portable or outside of the standard
ADD_CXX_DEFINITIONS("-Wall -Wextra") # Turn on warnings
ADD_CXX_DEFINITIONS("-Werror") # Treat warnings as errors
ADD_CXX_DEFINITIONS("-Wno-unknown-pragmas")
if (CMAKE_COMPILER_IS_GNUCXX AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER 9.0)
ADD_CXX_DEFINITIONS("-Wno-deprecated-copy")
Expand Down
4 changes: 2 additions & 2 deletions src/EnergyPlus/AirLoopHVACDOAS.cc
Original file line number Diff line number Diff line change
Expand Up @@ -930,14 +930,14 @@ namespace AirLoopHVACDOAS {
this->GetDesignDayConditions(state);

if (this->m_FanIndex > -1 && this->m_FanTypeNum == SimAirServingZones::Fan_System_Object) {
HVACFan::fanObjs[this->m_FanIndex]->designAirVolFlowRate = sizingMassFlow / DataEnvironment::StdRhoAir;
HVACFan::fanObjs[this->m_FanIndex]->designAirVolFlowRate = sizingMassFlow / state.dataEnvrn->StdRhoAir;
DataLoopNode::Node(this->m_FanInletNodeNum).MassFlowRateMaxAvail = sizingMassFlow;
DataLoopNode::Node(this->m_FanOutletNodeNum).MassFlowRateMaxAvail = sizingMassFlow;
DataLoopNode::Node(this->m_FanOutletNodeNum).MassFlowRateMax = sizingMassFlow;
}
bool errorsFound = false;
if (this->m_FanIndex > 0 && this->m_FanTypeNum == SimAirServingZones::Fan_ComponentModel) {
Fans::SetFanData(state, this->m_FanIndex, errorsFound, Name, sizingMassFlow / DataEnvironment::StdRhoAir, 0);
Fans::SetFanData(state, this->m_FanIndex, errorsFound, Name, sizingMassFlow / state.dataEnvrn->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
23 changes: 9 additions & 14 deletions src/EnergyPlus/AirflowNetwork/src/Solver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,6 @@ namespace AirflowNetwork {
// USE STATEMENTS:

// Using/Aliasing
using DataEnvironment::Latitude;
using DataEnvironment::OutBaroPress;
using DataEnvironment::OutDryBulbTemp;
using DataEnvironment::OutHumRat;
using DataEnvironment::StdBaroPress;
using DataSurfaces::Surface;

//std::vector<AirProperties> properties;
Expand Down Expand Up @@ -514,12 +509,12 @@ namespace AirflowNetwork {
}
// Compute zone air properties.
for (n = 1; n <= NetworkNumOfNodes; ++n) {
properties[n].density = AIRDENSITY(state, StdBaroPress + PZ(n), properties[n].temperature, properties[n].humidityRatio);
properties[n].density = AIRDENSITY(state, state.dataEnvrn->StdBaroPress + PZ(n), properties[n].temperature, properties[n].humidityRatio);
// RHOZ(n) = PsyRhoAirFnPbTdbW(StdBaroPress + PZ(n), TZ(n), WZ(n));
if (AirflowNetworkNodeData(n).ExtNodeNum > 0) {
properties[n].density = AIRDENSITY(state, StdBaroPress + PZ(n), OutDryBulbTemp, OutHumRat);
properties[n].temperature = OutDryBulbTemp;
properties[n].humidityRatio = OutHumRat;
properties[n].density = AIRDENSITY(state, state.dataEnvrn->StdBaroPress + PZ(n), state.dataEnvrn->OutDryBulbTemp, state.dataEnvrn->OutHumRat);
properties[n].temperature = state.dataEnvrn->OutDryBulbTemp;
properties[n].humidityRatio = state.dataEnvrn->OutHumRat;
}
properties[n].sqrtDensity = std::sqrt(properties[n].density);
properties[n].viscosity = 1.71432e-5 + 4.828e-8 * properties[n].temperature;
Expand Down Expand Up @@ -2024,13 +2019,13 @@ namespace AirflowNetwork {
Real64 CONV;

// FLOW:
RhoREF = AIRDENSITY(state, PSea, OutDryBulbTemp, OutHumRat);
RhoREF = AIRDENSITY(state, PSea, state.dataEnvrn->OutDryBulbTemp, state.dataEnvrn->OutHumRat);

CONV = Latitude * 2.0 * DataGlobalConstants::Pi() / 360.0;
CONV = state.dataEnvrn->Latitude * 2.0 * DataGlobalConstants::Pi() / 360.0;
G = 9.780373 * (1.0 + 0.0052891 * pow_2(std::sin(CONV)) - 0.0000059 * pow_2(std::sin(2.0 * CONV)));

Hfl = 1.0;
Pbz = OutBaroPress;
Pbz = state.dataEnvrn->OutBaroPress;
Nl = NumOfLinksMultiZone;
OpenNum = 0;
RhoLd(1) = 1.2;
Expand Down Expand Up @@ -2105,8 +2100,8 @@ namespace AirflowNetwork {
}

// RhoDrL is Rho at link level without pollutant but with humidity
RhoDrL(i, 1) = AIRDENSITY(state, OutBaroPress + PzFrom, TempL1, Xhl1);
RhoDrL(i, 2) = AIRDENSITY(state, OutBaroPress + PzTo, TempL2, Xhl2);
RhoDrL(i, 1) = AIRDENSITY(state, state.dataEnvrn->OutBaroPress + PzFrom, TempL1, Xhl1);
RhoDrL(i, 2) = AIRDENSITY(state, state.dataEnvrn->OutBaroPress + PzTo, TempL2, Xhl2);

// End initialisation

Expand Down
Loading

5 comments on commit f99ad62

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

6050_ReturnFan_MinFlowFrac (mitchute) - x86_64-MacOS-10.15-clang-11.0.0: OK (2992 of 2995 tests passed, 1 test warnings)

Messages:\n

  • 4 tests had: EIO diffs.
  • 1 test had: Table small diffs.
  • 3 tests had: ESO big diffs.
  • 3 tests had: Table big diffs.
  • 1 test had: ERR diffs.

Failures:\n

regression Test Summary

  • Passed: 713
  • Failed: 3

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

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

6050_ReturnFan_MinFlowFrac (mitchute) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (3031 of 3035 tests passed, 1 test warnings)

Messages:\n

  • 5 tests had: EIO diffs.
  • 1 test had: Table small diffs.
  • 3 tests had: ESO big diffs.
  • 4 tests had: Table big diffs.
  • 1 test had: ERR diffs.
  • 1 test had: ESO small diffs.
  • 1 test had: MTR small diffs.

Failures:\n

regression Test Summary

  • Passed: 732
  • Failed: 4

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.

6050_ReturnFan_MinFlowFrac (mitchute) - Win64-Windows-10-VisualStudio-16: OK (2252 of 2252 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

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

6050_ReturnFan_MinFlowFrac (mitchute) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1561 of 1561 tests passed, 0 test warnings)

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.

6050_ReturnFan_MinFlowFrac (mitchute) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: Coverage Too Low

Failures:\n

integration Test Summary

  • Passed: 720
  • Timeout: 1

Build Badge Test Badge Coverage Badge

Please sign in to comment.