Skip to content

Commit

Permalink
Merge remote-tracking branch 'remotes/origin/develop' into basementNAN
Browse files Browse the repository at this point in the history
  • Loading branch information
mjwitte committed Mar 31, 2022
2 parents f31be95 + ed759b1 commit b75e876
Show file tree
Hide file tree
Showing 5 changed files with 33 additions and 11 deletions.
6 changes: 5 additions & 1 deletion cmake/CompilerFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -75,11 +75,15 @@ elseif(CMAKE_COMPILER_IS_GNUCXX OR "${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang" O
target_compile_options(project_warnings INTERFACE -Wno-delete-non-virtual-dtor)
target_compile_options(project_warnings INTERFACE -Wno-missing-braces)
if(CMAKE_COMPILER_IS_GNUCXX) # g++
target_compile_options(project_warnings INTERFACE -Wno-unused-but-set-parameter -Wno-unused-but-set-variable)
# Suppress unused-but-set warnings until more serious ones are addressed
target_compile_options(project_warnings INTERFACE -Wno-unused-but-set-parameter -Wno-unused-but-set-variable)
target_compile_options(project_warnings INTERFACE -Wno-maybe-uninitialized)
target_compile_options(project_warnings INTERFACE -Wno-aggressive-loop-optimizations)
elseif("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang" OR "${CMAKE_CXX_COMPILER_ID}" STREQUAL "AppleClang")
if(CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 13.0)
# Suppress unused-but-set warnings until more serious ones are addressed
target_compile_options(project_warnings INTERFACE -Wno-unused-but-set-parameter -Wno-unused-but-set-variable)
endif()
target_compile_options(project_warnings INTERFACE -Wno-vexing-parse)
target_compile_options(project_warnings INTERFACE -Wno-invalid-source-encoding)
endif()
Expand Down
12 changes: 6 additions & 6 deletions release/readme.html
Original file line number Diff line number Diff line change
Expand Up @@ -200,8 +200,8 @@ <h2><a id="Multiple_Version_Conversions_51"></a>Multiple Version Conversions</h2
version (including .rvi and .mvi files) to the current version. The install also includes the necessary programs
to convert from older versions back to v7.2.0. For versions older than v7.2.0, first we applaud your dedication
to EnergyPlus that stretches for quite some time, and second you will need to visit the
helpdesk (<a href="https://energyplushelp.freshdesk.com/">energyplushelp.freshdesk.com</a>) to request multiple
conversion set of programs.
<a href="https://energyplushelp.freshdesk.com/support/solutions/folders/67000241428">Helpdesk</a> to request
multiple conversion set of programs.
</p>

<h2><a id="Documentation_55"></a>Documentation</h2>
Expand Down Expand Up @@ -230,7 +230,7 @@ <h2><a id="Weather_Data_63"></a>Weather Data</h2>
EnergyPlus uses these files directly for annual simulations. The Weather Converter program that is part of
EnergyPlus can read these weather files and provide .csv (comma separated variable) as well as statistics
reports
about the included (or other) files. Weather data for more than 2100 locations in more than 100 countries can be
about the included (or other) files. Weather data for more than 3000 locations in more than 100 countries can be
downloaded on the <a href="https://energyplus.net/weather">EnergyPlus Weather Page</a>.</p>

<h2>Highlights of this release (V22.1.0)</h2>
Expand Down Expand Up @@ -386,13 +386,13 @@ <h2><a id="Platforms_90"></a>Platforms</h2>
Packages changed slightly as of the 9.4 release.
We began building our packages on a new system which adds better support
for newer platforms. During the change, we stepped away from some older OS versions. For example, we are
not testing on Windows 7 or 8, only 10. However, both our 32-bit and 64-bit packages now include the new
not testing on Windows 7 or 8, only 10 and 11. However, both our 32-bit and 64-bit packages now include the new
Python plugin system, rather than just the 64-bit as previously. We have also added packaging and testing
with Ubuntu 20.04 <i>in addition to</i> Ubuntu 18.04. On Mac, we are officially releasing an installer for
OSX 10.15 and moving toward OSX 11 as well.
</p>
<ul class="list-group">
<li class="list-group-item">Windows 10</li>
<li class="list-group-item">Windows 10 and 11</li>
<li class="list-group-item">Linux (Ubuntu 18.04 and 20.04) 64 bit versions</li>
<li class="list-group-item">Mac OSX 10.15 64 bit versions</li>
<li class="list-group-item">EnergyPlus V22.1 has been tested on all of these platforms</li>
Expand All @@ -413,7 +413,7 @@ <h2><a id="Known_problems_104"></a>Known problems</h2>
known (and even more unknown) probably remain. Specifics on the remaining known problems can be found on the
GitHub repository <a href="https://github.com/NREL/EnergyPlus/issues">issues page</a>. If you are interested in
what has been addressed, the issues resolved in this release are installed similarly in the
<a href="changelog.html">changelog</a>.
<a href="https://github.com/NREL/EnergyPlus/releases/latest">release details</a>.
</p>

<h2>Additional steps you may want to consider:</h2>
Expand Down
20 changes: 18 additions & 2 deletions src/EnergyPlus/PluginManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -516,8 +516,9 @@ PluginManager::PluginManager(EnergyPlusData &state) : eplusRunningViaPythonAPI(s
state,
"PluginManager: Search path inputs requested adding epin variable to Python path, but epin variable was empty, skipping.");
} else {
if (FileSystem::pathExists(epinPathObject)) {
fs::path sanitizedEnvInputDir = PluginManager::sanitizedPath(epinPathObject);
auto epinRootDir = FileSystem::getParentDirectoryPath(fs::path(epinPathObject));
if (FileSystem::pathExists(epinRootDir)) {
fs::path sanitizedEnvInputDir = PluginManager::sanitizedPath(epinRootDir);
PluginManager::addToPythonPath(state, sanitizedEnvInputDir, true);
} else {
EnergyPlus::ShowWarningMessage(state,
Expand All @@ -541,6 +542,21 @@ PluginManager::PluginManager(EnergyPlusData &state) : eplusRunningViaPythonAPI(s
// nothing to do here
}
}
} else {
// if no search path objects exist, we still need to do the default searching
PluginManager::addToPythonPath(state, ".", false);
fs::path sanitizedInputFileDir = PluginManager::sanitizedPath(state.dataStrGlobals->inputDirPath);
PluginManager::addToPythonPath(state, sanitizedInputFileDir, false);
std::string epin_path;
get_environment_variable("epin", epin_path);
fs::path epinPathObject = fs::path(epin_path);
if (!epinPathObject.empty()) {
auto epinRootDir = FileSystem::getParentDirectoryPath(fs::path(epinPathObject));
if (FileSystem::pathExists(epinRootDir)) {
fs::path sanitizedEnvInputDir = PluginManager::sanitizedPath(epinRootDir);
PluginManager::addToPythonPath(state, sanitizedEnvInputDir, true);
}
}
}

// Now read all the actual plugins and interpret them
Expand Down
5 changes: 3 additions & 2 deletions third_party/kiva-ep/src/libkiva/Ground.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,8 @@ void Ground::calculateSurfaceAverages() {
}

double totalQ = 0.0;
double totalQc = 0.0, totalQr = 0.0;
double totalQc = 0.0;
// double totalQr = 0.0;
double TA = 0;
double hA = 0.0, hcA = 0.0, hrA = 0.0;
double totalArea = 0.0;
Expand Down Expand Up @@ -368,7 +369,7 @@ void Ground::calculateSurfaceAverages() {
hA += Ahc + Ahr;

totalQc += Qc;
totalQr += Qr;
// totalQr += Qr;
totalQ += Qc + Qr + q * A;

TA += TNew[index] * A;
Expand Down
1 change: 1 addition & 0 deletions third_party/kiva-ep/src/libkiva/Ground.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
#pragma clang diagnostic push
#if __clang_major__ >= 13
#pragma clang diagnostic ignored "-Wdeprecated-copy"
#pragma clang diagnostic ignored "-Wdeprecated-declarations"
#endif
#elif defined(__GNUC__) && defined(__linux__)
#pragma GCC diagnostic push
Expand Down

4 comments on commit b75e876

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

basementNAN (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2486 of 2486 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.

basementNAN (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (3270 of 3270 tests passed, 0 test warnings)

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.

basementNAN (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1744 of 1744 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.

basementNAN (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: OK (746 of 746 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.