Smart-City-Sample icon indicating copy to clipboard operation
Smart-City-Sample copied to clipboard

Published 20 hours ago • verified publisher icon OpenVisualCloud

Latest update shows problem with recognition and reidentification models

Open divdaisymuffin opened this issue 3 years ago • 5 comments

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)

divdaisymuffin avatar Jul 07 '21 04:07 divdaisymuffin

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

xwu2git avatar Jul 07 '21 21:07 xwu2git

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

xwu2git avatar Jul 08 '21 01:07 xwu2git

any update about it?

divdaisymuffin avatar Jul 19 '21 10:07 divdaisymuffin

@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 avatar Jul 26 '21 13:07 nnshah1

@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: https://github.com/OpenVisualCloud/Smart-City-Sample/pull/783

nnshah1 avatar Jul 27 '21 16:07 nnshah1