New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix glass U factor sql error #4681
Conversation
std::string query = R"(SELECT Value from TabularDataWithStrings | ||
WHERE ReportName = 'EnvelopeSummary' | ||
AND ReportForString = 'Entire Facility' | ||
AND TableName = 'Opaque Exterior' | ||
AND ColumnName = 'U-Factor with Film' | ||
AND Units='W/m2-K' | ||
AND RowName = ?;)"; | ||
outputResult = sqlFile->execAndReturnFirstDouble(query, to_upper_copy(*subSurfaceName)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 smart!
src/model/test/Surface_GTest.cpp
Outdated
|
||
ASSERT_TRUE(surface->uFactor()); | ||
double uFactor = surface->uFactor().get(); | ||
EXPECT_TRUE(openstudio::equal(0.310, uFactor, 1.0E-3)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EXPECT_DOUBLE_EQ(val1, val2)
is meant for this, and will report a nicer output in case it fails.
Verifies that the two double values val1 and val2 are approximately equal, to within 4 ULPs from each other.
If you need a specific tolerance (I don't think you do), then use EXPECT_NEAR(val1, val2, abs_error)
CI Results for 294ca1d:
|
Pull request overview
Pull Request Author
src/model/test
)src/energyplus/Test
)src/osversion/VersionTranslator.cpp
)Labels:
IDDChange
APIChange
Pull Request - Ready for CI
so that CI builds your PRReview Checklist
This will not be exhaustively relevant to every PR.