IntenseWebs/openvino
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove the assert that could terminate the process, throw an exception instead (#3205)

Browse Source
This commit is contained in:
Tomasz Dołbniak 2020-11-18 20:27:51 +01:00 committed by GitHub
parent 39ec38383a
commit a1d858c502
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ngraph/test/util/engine/interpreter_engine.cpp
View File

@ -98,7 +98,10 @@ namespace
// TODO: add testing infrastructure for float16 and bfloat16 to avoid cast to double // TODO: add testing infrastructure for float16 and bfloat16 to avoid cast to double
std::vector<double> expected_double(expected.size()); std::vector<double> expected_double(expected.size());
std::vector<double> result_double(result.size()); std::vector<double> result_double(result.size());
assert(expected.size() == result.size() && "expected and result size must match");
NGRAPH_CHECK(expected.size() == result.size(),
"Number of expected and computed results don't match");
for (int i = 0; i < expected.size(); ++i) for (int i = 0; i < expected.size(); ++i)
{ {
expected_double[i] = static_cast<double>(expected[i]); expected_double[i] = static_cast<double>(expected[i]);
@ -107,7 +110,7 @@ namespace
return ngraph::test::all_close_f(expected_double, result_double, tolerance_bits); return ngraph::test::all_close_f(expected_double, result_double, tolerance_bits);
} }
}; }; // namespace
test::INTERPRETER_Engine::INTERPRETER_Engine(const std::shared_ptr<Function> function) test::INTERPRETER_Engine::INTERPRETER_Engine(const std::shared_ptr<Function> function)
: m_function{function} : m_function{function}