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

Latest update shows problem with recognition and reidentification models #778

Open
divdaisymuffin opened this issue Jul 7, 2021 · 5 comments

Comments

@divdaisymuffin
Copy link

Hi @xwu2git,
We recently encountered few issues with latest update of Smart-City-Sample:

  1. The entrance part does not work and shows error with Person-reidentification model.
  2. The emotion recognition model when put in pipeline, also shows same error.
  3. Both were working fine with 2-3 month old version.

For reference, I am adding screenshots of both.

Screenshot (86)

Screenshot (85)

@xwu2git
Copy link
Collaborator

xwu2git commented Jul 7, 2021

Reproduced. Looks like a model mismatch to the latest OpenVINO version. Investigating.

@xwu2git
Copy link
Collaborator

xwu2git commented Jul 8, 2021

@QuPengfei @nnshah1, Can you take a look? I tried to upgrade to different versions in the OpenVINO model zoo models. None of them works.

@divdaisymuffin
Copy link
Author

any update about it?

@nnshah1
Copy link

nnshah1 commented Jul 26, 2021

@divdaisymuffin , @xwu2git

I took a quick look and this seems related to the INT8 versions of the model specifically. Downloading and converting the models with the latest public VAServing dependencies does seem to work but those same models do not work in the latest tip of openvisualcloud. I'll continue looking to see if a compatible INT8 model can be found, in the mean time from a functional perspective you can try removing the INT8 models temporarily and fallback to FP32.

@nnshah1
Copy link

nnshah1 commented Jul 27, 2021

@divdaisymuffin, @xwu2git I did some more digging and found the issue may be related to the model proc file and changes in DL Streamer. Can you check to see if this pull request fixes the issue for you: #783

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

3 participants