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

Migrate examples to use depthai 2.0.0.1 and follow a common blob naming format #96

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

VanDavv
Copy link
Contributor

@VanDavv VanDavv commented Mar 15, 2021

While migrating examples to the latest PyPi release, and making sure the blobs are using the latest OpenVINO and have correct blob names, I found two examples that stopped working as they used to:

  • Pedestrian Reidentification network (person-reidentification-retail-0031) is no longer available in 2021.2, was removed in 2020.4. The new network that I tried to use - person_reidentification_retail_0288 and person_reidentification_retail_0277 - accept an input image in higher resolution (now 128x256, previously 48x96), which may be a cause of a significant inference speed drop when these networks start to process the frames.

    Code for this example is here as was not included in this PR due to the performance issues

  • Tensorflow Image Classification example has accuracy issues when model is changed to the one compiled for 6 or 8 shaves. Where previously is recognized a dandelion (like on a demo image) it does no more. That's why this example is also not included in this PR, and its source code is here

Other than these two, the rest of the examples work fine.

@SzabolcsGergely
Copy link
Contributor

You can use openvino IR model compiled for 2020.1/3/4 for person-reidentification-retail-0031 and use myriad compiler from 2021.2, it will likely work. That's how I compiled deeplabv3 for openvino 2021.2 since I had only 2020.1 IR model from gen1. @VanDavv

@VanDavv
Copy link
Contributor Author

VanDavv commented Mar 15, 2021

That's very clever, will do. Thanks @szabi-luxonis !

Copy link
Contributor

@SzabolcsGergely SzabolcsGergely left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@VanDavv VanDavv merged commit 5c2f87b into master Mar 16, 2021
@VanDavv VanDavv deleted the update_depthai_version branch March 16, 2021 13:47
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

Successfully merging this pull request may close these issues.

None yet

2 participants