search

openvinotoolkit / openvino

OpenVINO™ is an open-source toolkit for optimizing and deploying AI inference
https://docs.openvino.ai
Apache License 2.0
6.79k stars 2.16k forks source link

Missing U32 precision #1292

Closed adk9 closed 4 years ago

adk9 commented 4 years ago

I have a nGraph constant node with element type as u32 in an nGraph function that I'm using to create a CNNNetwork. When trying to load the network to the CPU plugin, I see the following error:

ie_ngraph_utils.hpp:117  Incorrect precision u32!

Looking at the API documentation here, it looks like there's no InferenceEngine::Precision::U32. Is U32 precision type missing?

ilyachur commented 4 years ago

Yes at the current moment IE doesn't contain U32 precision. Due to this, we cannot convert nGraph u32 to Inference Engine precision

ilyachur commented 4 years ago

@adk9 I added U32 precision in the next PR: https://github.com/openvinotoolkit/openvino/pull/1297

Please check that it works for your case.

adk9 commented 4 years ago

@ilyachur Thanks for the quick PR. These changes work for my case and I'm now able to pass nGraph functions with ops having U32 precision.

ilyachur commented 4 years ago

@adk9 Cool! The PR was merged to the master branch.

Please try to use the master branch and please close the issue if all is ok.

adk9 commented 4 years ago

Everything looks good on master; closing the issue now.