search

OpenVisualCloud / Smart-City-Sample

The smart city reference pipeline shows how to integrate various media building blocks, with analytics powered by the OpenVINO™ Toolkit, for traffic or stadium sensing, analytics and management tasks.
BSD 3-Clause "New" or "Revised" License
189 stars 81 forks source link

Latest update shows problem with recognition and reidentification models #778

Open divdaisymuffin opened 3 years ago

divdaisymuffin commented 3 years ago

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 commented 3 years ago

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

xwu2git commented 3 years ago

@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 commented 3 years ago

any update about it?

nnshah1 commented 3 years ago

@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 commented 3 years ago

@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