Skip to content
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

Is there any format change between OpenVINO R3 and R4? #127

Closed
chaoli2 opened this issue Apr 12, 2019 · 5 comments
Closed

Is there any format change between OpenVINO R3 and R4? #127

chaoli2 opened this issue Apr 12, 2019 · 5 comments

Comments

@chaoli2
Copy link

chaoli2 commented Apr 12, 2019

I use OpenVINO for object detection in my project. I convert yolov2-voc model to OpenVINO IR, and it works well under OpenVINO R3.

However, when I upgrade OpenVINO R3 to R4, I received [ ERROR ] Dims and format are inconsistent. I've checked that my code and model files are not changed. And I've spent some time to locate the root cause but no clue yet. Interface CreateInferRequestPtr() is the entrance in my project which leads to this issue. It is defined in class "InferenceEngine::ExecutableNetwork ". I also found that this exception is caused here. But I have no idea why. I'd like to know what changes between R3 and R4 will cause this issue and how I could update my project to fix it. Is there anyone can help? Thanks very much.

@shubha-ramani
Copy link

Dear @chaoli2

Is there any reason you're using such an ancient version of OpenVino ? Please upgrade your project to the latest version, 2019 R1.

Thanks,

Shubha

@chaoli2
Copy link
Author

chaoli2 commented Apr 15, 2019

@shubha-ramani
Thanks for your advice. Actually upgrading OpenVINO is exactly what we want to do. This issue blocks the upgrading process.
At the beginning, our project was developed under OpenVINO R3, so we first verified that there is no such kind issue at that time. When we tried to upgrade OpenVINO R4, even OpenVINO R5, we received this [ ERROR ] Dims and format are inconsistent. Currently we have no idea what we can do fix it.

@chaoli2
Copy link
Author

chaoli2 commented Apr 15, 2019

Is there official sample code for yolov2? I only found yolov3 samples inside the OpenVINO package. I have to refer to community projects, like this. Sometimes they may be out of date due to lack of maintenance.

@shubha-ramani
Copy link

Dear @chaoli2

Unfortunately we do not have yolo v2 samples. We only have yolo v3 (both Python and C++ versions).
To answer your question about the format change between OpenVino R3 and R4, here is what I suggest. It's a bit laborious but if you follow this method you will likely answer your question.

  1. clone the R4 version of dldt
  2. Follow the below doc to build a DEBUG version of Inference Engine
    https://github.com/opencv/dldt/blob/2018/inference-engine/README.md
    Make sure you build a debug version of your inference app also
  3. Step through the code using your favorite debugger

If you follow these steps you will find out the exact layer which is causing that error. I have done this before myself and it's quite effective.

Hope you will give it a try !

Thanks for using OpenVino,

Shubha

@chaoli2
Copy link
Author

chaoli2 commented Apr 24, 2019

Thanks for your information. I've found the cause for [ ERROR ] Dims and format are inconsistent.

When initializing output ptr, there is no need to output_data_ptr->setLayout(InferenceEngine::Layout::NCHW) but I did.

Everything goes well after I remove above code. The only question now is why this error cannot be reproduced under OpenVINO R3.

@chaoli2 chaoli2 closed this as completed Apr 24, 2019
eshoguli pushed a commit to eshoguli/openvino that referenced this issue Jun 1, 2021
alvoron added a commit to alvoron/openvino that referenced this issue Apr 4, 2023
alvoron added a commit to alvoron/openvino that referenced this issue Apr 11, 2023
popovaan added a commit to popovaan/openvino that referenced this issue Sep 8, 2023
* Examples code correction.

* Change format of example.

* Conflict fix.

* Remove wrong change.

* Added input_shapes example.

* batch example.

* Examples format changed.

* List item removed.

* Remove list for all examples.

* Corrected batch example.

* Transform example.

* Text corrections.

* Text correction.

* Example correction.

* Small correction.

* Small correction.

* Small correction.

* Small correction.

* Text corrections.

* Links corrected.
slyalin added a commit that referenced this issue Sep 12, 2023
* Added OVC and ov.convert_model() description.

* Minor corrections.

* Small correction.

* Include page to toctree.

* WIP: Model Preparation

* Forked OVC/ov.convert_model documentation sub-directory; reworked model_introduction.md

* Reverted ovc-related changes in old MO_DG documentation

* State explicitly that MO is considered legacy API

* Reduced ovc description in model preparation part; added TF Hub exampe (via file)

* Grammar check; removed obsolexte parts not relevant to ovc; better wording

* Removed a duplicate of mo-to-ovc transition

* Fixed links and some other errors found in documentation build

* Resolved XYZ placeholder to the transition guide

* Fixed technical issues with links

* Up-to-date link to PTQ chapter (instead of obsolete POT)

* Fixed strong text ending

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Renamed Legacy conversion guides

* Fixed links and styles for inlined code

* Fixed style for code references

* Fixing technical syntax errors in docs

* Another attempt to fix docs

* Removed all unreferenced images

* Better content for Additional Resources in model preporation introduction

* MO to OVC transition guide. (#127)

* Examples code correction.

* Change format of example.

* Conflict fix.

* Remove wrong change.

* Added input_shapes example.

* batch example.

* Examples format changed.

* List item removed.

* Remove list for all examples.

* Corrected batch example.

* Transform example.

* Text corrections.

* Text correction.

* Example correction.

* Small correction.

* Small correction.

* Small correction.

* Small correction.

* Text corrections.

* Links corrected.

* Text corrections (#128)

* Text corrections.

* Example corrected.

* Update docs/install_guides/pypi-openvino-dev.md

Co-authored-by: Sergey Lyalin <[email protected]>

---------

Co-authored-by: Sergey Lyalin <[email protected]>

* Many technical fixes, description of recursive flattening of list and tuples

* Reorganized structure of Model Conversion toc tree. Removed fp16 dedicated page, merged to Conversion Parameters.

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_TensorFlow.md

Co-authored-by: Roman Kazantsev <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Maciej Smyk <[email protected]>

* Fixed example from tf hub. Removed input_shape references

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Removed

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Fixed links

* Removed TODO for model flow

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Restored lost code-blocks that leaded to wrong rendering of the code snippets in some places

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

* Fixed links to notebooks

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

---------

Co-authored-by: Anastasiia Pnevskaia <[email protected]>
Co-authored-by: Karol Blaszczak <[email protected]>
Co-authored-by: Roman Kazantsev <[email protected]>
Co-authored-by: Maciej Smyk <[email protected]>
Co-authored-by: Tatiana Savina <[email protected]>
kblaszczak-intel referenced this issue in kblaszczak-intel/openvino Sep 12, 2023
* Added OVC and ov.convert_model() description.

* Minor corrections.

* Small correction.

* Include page to toctree.

* WIP: Model Preparation

* Forked OVC/ov.convert_model documentation sub-directory; reworked model_introduction.md

* Reverted ovc-related changes in old MO_DG documentation

* State explicitly that MO is considered legacy API

* Reduced ovc description in model preparation part; added TF Hub exampe (via file)

* Grammar check; removed obsolexte parts not relevant to ovc; better wording

* Removed a duplicate of mo-to-ovc transition

* Fixed links and some other errors found in documentation build

* Resolved XYZ placeholder to the transition guide

* Fixed technical issues with links

* Up-to-date link to PTQ chapter (instead of obsolete POT)

* Fixed strong text ending

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Renamed Legacy conversion guides

* Fixed links and styles for inlined code

* Fixed style for code references

* Fixing technical syntax errors in docs

* Another attempt to fix docs

* Removed all unreferenced images

* Better content for Additional Resources in model preporation introduction

* MO to OVC transition guide. (#127)

* Examples code correction.

* Change format of example.

* Conflict fix.

* Remove wrong change.

* Added input_shapes example.

* batch example.

* Examples format changed.

* List item removed.

* Remove list for all examples.

* Corrected batch example.

* Transform example.

* Text corrections.

* Text correction.

* Example correction.

* Small correction.

* Small correction.

* Small correction.

* Small correction.

* Text corrections.

* Links corrected.

* Text corrections (#128)

* Text corrections.

* Example corrected.

* Update docs/install_guides/pypi-openvino-dev.md

Co-authored-by: Sergey Lyalin <[email protected]>

---------

Co-authored-by: Sergey Lyalin <[email protected]>

* Many technical fixes, description of recursive flattening of list and tuples

* Reorganized structure of Model Conversion toc tree. Removed fp16 dedicated page, merged to Conversion Parameters.

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_TensorFlow.md

Co-authored-by: Roman Kazantsev <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Maciej Smyk <[email protected]>

* Fixed example from tf hub. Removed input_shape references

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Removed

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Fixed links

* Removed TODO for model flow

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Restored lost code-blocks that leaded to wrong rendering of the code snippets in some places

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

* Fixed links to notebooks

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

---------

Co-authored-by: Anastasiia Pnevskaia <[email protected]>
Co-authored-by: Karol Blaszczak <[email protected]>
Co-authored-by: Roman Kazantsev <[email protected]>
Co-authored-by: Maciej Smyk <[email protected]>
Co-authored-by: Tatiana Savina <[email protected]>
yangwang201911 pushed a commit to yangwang201911/openvino that referenced this issue Sep 13, 2023
* Added OVC and ov.convert_model() description.

* Minor corrections.

* Small correction.

* Include page to toctree.

* WIP: Model Preparation

* Forked OVC/ov.convert_model documentation sub-directory; reworked model_introduction.md

* Reverted ovc-related changes in old MO_DG documentation

* State explicitly that MO is considered legacy API

* Reduced ovc description in model preparation part; added TF Hub exampe (via file)

* Grammar check; removed obsolexte parts not relevant to ovc; better wording

* Removed a duplicate of mo-to-ovc transition

* Fixed links and some other errors found in documentation build

* Resolved XYZ placeholder to the transition guide

* Fixed technical issues with links

* Up-to-date link to PTQ chapter (instead of obsolete POT)

* Fixed strong text ending

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/MO_OVC_transition.md

Co-authored-by: Anastasiia Pnevskaia <[email protected]>

* Renamed Legacy conversion guides

* Fixed links and styles for inlined code

* Fixed style for code references

* Fixing technical syntax errors in docs

* Another attempt to fix docs

* Removed all unreferenced images

* Better content for Additional Resources in model preporation introduction

* MO to OVC transition guide. (openvinotoolkit#127)

* Examples code correction.

* Change format of example.

* Conflict fix.

* Remove wrong change.

* Added input_shapes example.

* batch example.

* Examples format changed.

* List item removed.

* Remove list for all examples.

* Corrected batch example.

* Transform example.

* Text corrections.

* Text correction.

* Example correction.

* Small correction.

* Small correction.

* Small correction.

* Small correction.

* Text corrections.

* Links corrected.

* Text corrections (openvinotoolkit#128)

* Text corrections.

* Example corrected.

* Update docs/install_guides/pypi-openvino-dev.md

Co-authored-by: Sergey Lyalin <[email protected]>

---------

Co-authored-by: Sergey Lyalin <[email protected]>

* Many technical fixes, description of recursive flattening of list and tuples

* Reorganized structure of Model Conversion toc tree. Removed fp16 dedicated page, merged to Conversion Parameters.

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_TensorFlow.md

Co-authored-by: Roman Kazantsev <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Maciej Smyk <[email protected]>

* Fixed example from tf hub. Removed input_shape references

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

Co-authored-by: Tatiana Savina <[email protected]>

* Removed

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/OV_Converter_UG/prepare_model/convert_model/Convert_Model_From_ONNX.md

Co-authored-by: Tatiana Savina <[email protected]>

* Fixed links

* Removed TODO for model flow

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Restored lost code-blocks that leaded to wrong rendering of the code snippets in some places

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

* Update docs/Documentation/model_introduction.md

* Fixed links to notebooks

* Apply suggestions from code review

Co-authored-by: Tatiana Savina <[email protected]>

---------

Co-authored-by: Anastasiia Pnevskaia <[email protected]>
Co-authored-by: Karol Blaszczak <[email protected]>
Co-authored-by: Roman Kazantsev <[email protected]>
Co-authored-by: Maciej Smyk <[email protected]>
Co-authored-by: Tatiana Savina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants