openvinotoolkit / open_model_zoo

Pre-trained Deep Learning models and demos (high quality and extremely fast)
https://docs.openvino.ai/latest/model_zoo.html
Apache License 2.0
4.1k stars 1.37k forks source link

Following layers are not supported by the plugin for specified device CPU #577

Closed sss22213 closed 5 years ago

sss22213 commented 5 years ago
vladimir-dudnik commented 5 years ago

@sss22213 you have mixed detector and reid models. Detector model should be person-detection-retail-0013 and reid model should be person-reidentification-retail-0079

IRDonch commented 5 years ago

@vladimir-dudnik is right, and the demo does work if you swap the options. However, I'd say there's still a bug in the demo: the CPU extensions are only used for the detector model and not for the reid model.

vladimir-dudnik commented 5 years ago

@IRDonch I do not see problem here. Person reidentification models do not use extended layers and so cpu_extension library is not required for them.

IRDonch commented 5 years ago

Person reidentification models do not use extended layers

Ours don't, but the user could, in principle, use their own interface-compatible model instead. I think it's a reasonable use case.

vladimir-dudnik commented 5 years ago

@IRDonch too much assumptions. We have listed models which this demo was tested on in models.lst file. And we can't guarantee that other models will work without changing the demo. So, anyone who want to add their model to this demo have to ensure it work and made necessary changes if needed (that might be not only loading of extended layers)

sss22213 commented 5 years ago

thank you, i sovled.