Skip to content

TESTS: Added test for parallel LoadNetwork with accuracy check#858

Merged
ilya-lavrenov merged 1 commit intoopenvinotoolkit:releases/2020/4from
ilya-lavrenov:load-network-paralle-accuracy-test
Jun 12, 2020
Merged

TESTS: Added test for parallel LoadNetwork with accuracy check#858
ilya-lavrenov merged 1 commit intoopenvinotoolkit:releases/2020/4from
ilya-lavrenov:load-network-paralle-accuracy-test

Conversation

@ilya-lavrenov
Copy link
Copy Markdown
Contributor

@ilya-lavrenov ilya-lavrenov added the category: IE Tests OpenVINO Test: plugins and common label Jun 10, 2020
@ilya-lavrenov ilya-lavrenov added this to the 2020.4 milestone Jun 10, 2020
@ilya-lavrenov ilya-lavrenov requested review from a team, iefode and vladimir-paramuzov June 10, 2020 11:47
@ilya-lavrenov ilya-lavrenov force-pushed the load-network-paralle-accuracy-test branch from 756189a to 8694187 Compare June 10, 2020 13:38
@ilya-lavrenov
Copy link
Copy Markdown
Contributor Author

@vladimir-paramuzov could you please have a look why the tests fail on GPU?

@vladimir-paramuzov
Copy link
Copy Markdown

@vladimir-paramuzov could you please have a look why the tests fail on GPU?

@ilya-lavrenov As I understand, the issue is connected with the lifetime of output blob from the test run. CLDNN plugin just wrap up the memory allocated by cldnn::network for each output blob to avoid extra copies, so once ExecNetwork is freed, these memory buffers are invalidated as well. The solution is to allocate unique output buffers for each infer request and force cldnn to use them instead of ones allocated internally. This is a known limitation in cldnn and we have a task to fix it (27643).

So as a WA I can suggest either temporary suppression of these tests for GPU or using SetBlob API:

req.SetInput(blobs);
InferenceEngine::Blob::Ptr blob = make_blob_with_precision(network.getOutputsInfo().begin()->second->getTensorDesc());
blob->allocate();
req.SetBlob(network.getOutputsInfo().begin()->first, blob);
req.Infer();
return blob;

With these changes tests passed on my machine.

@ilya-lavrenov ilya-lavrenov force-pushed the load-network-paralle-accuracy-test branch from 8694187 to 401258d Compare June 10, 2020 22:08
@ilya-lavrenov
Copy link
Copy Markdown
Contributor Author

@vladimir-paramuzov thanks for the detailed explanations! Updated the test, hope GPU will pass.

@ilya-lavrenov ilya-lavrenov force-pushed the load-network-paralle-accuracy-test branch 4 times, most recently from c1a6399 to bafcea4 Compare June 11, 2020 22:11
@ilya-lavrenov ilya-lavrenov merged commit 259a4c2 into openvinotoolkit:releases/2020/4 Jun 12, 2020
@ilya-lavrenov ilya-lavrenov deleted the load-network-paralle-accuracy-test branch June 12, 2020 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: IE Tests OpenVINO Test: plugins and common

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants