diff options
author | Lev Proleev <levp@google.com> | 2020-05-05 14:11:23 +0100 |
---|---|---|
committer | David Gross <dgross@google.com> | 2020-05-05 17:06:07 +0000 |
commit | 62ef1032d5759c87ef4934202cd7e04918104414 (patch) | |
tree | 5f430b2c8f3ce6a4a73ef836113f1676f4ff38f5 /nn/runtime/test/TestGenerated.cpp | |
parent | 55bee1c40e56f690de39bae8e47a4b6f9743086c (diff) | |
download | ml-62ef1032d5759c87ef4934202cd7e04918104414.tar.gz |
Fix caching bug in quantization coupling test
Execute function in our tests among other things makes sure that the
models work with caching enabled. Caching token and directory are
constant and therefore the same for all tests. This is usually not a
problem since the tests check only one model with a set of different
configs but this was not the case for QuantizationCouplingTest suite.
This suite ran signed and unsigned models in one test making them share
the same caching token and directory which is an incorrect use of
caching API.
This CL solves this problem by removing the execution of an unsigned
model from the test. This is safe since the unsigned model is still
verified in GeneratedTests suite.
Bug: 155604227
Test: NNTest_static
Change-Id: I6aa2c8cd3fa41e37a5de53259d1fcc784f327093
Diffstat (limited to 'nn/runtime/test/TestGenerated.cpp')
-rw-r--r-- | nn/runtime/test/TestGenerated.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/nn/runtime/test/TestGenerated.cpp b/nn/runtime/test/TestGenerated.cpp index cc0b61983..a76aecd92 100644 --- a/nn/runtime/test/TestGenerated.cpp +++ b/nn/runtime/test/TestGenerated.cpp @@ -382,7 +382,6 @@ TEST_P(GeneratedValidationTests, Test) { } TEST_P(QuantizationCouplingTest, Test) { - execute(testModel); execute(convertQuant8AsymmOperandsToSigned(testModel)); } |