Skip to content

Fix G-API smart classroom demo#3881

Merged
Wovchena merged 4 commits intoopenvinotoolkit:masterfrom
smirnov-alexey:as/fix_gapi_smart_classroom_demo
Dec 19, 2023
Merged

Fix G-API smart classroom demo#3881
Wovchena merged 4 commits intoopenvinotoolkit:masterfrom
smirnov-alexey:as/fix_gapi_smart_classroom_demo

Conversation

@smirnov-alexey
Copy link
Copy Markdown
Contributor

@smirnov-alexey smirnov-alexey commented Dec 4, 2023

Demo is converted to G-API OV 2.0 backend which solves the problem with assumed NCHW layout vs NHWC which we get from the network (IRv11)

@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Dec 4, 2023

@Wovchena @TolyaTalamanov @dmatveev

@dmatveev
Copy link
Copy Markdown
Contributor

dmatveev commented Dec 4, 2023

@smirnov-alexey @TolyaTalamanov 🤷‍♂️

[ INFO ] The Face Detection model/tmp/tmp57ndv906/models/intel/face-detection-adas-0001/FP16/face-detection-adas-0001.xml is loaded to CPU device.
[ WARNING ] Face recognition models are disabled!
[ WARN:0@0.092] global govbackend.cpp:780 cfgLayouts Failed to find layout for input layer "data" - NCHW is set by default
[ ERROR ] Failed to copy data from cv::Mat to ov::Tensor. Data type or number of elements mismatch. ov::Tensor: u8 [1,480,640,680] and cv::Mat: 8UC3 640x480
Exit code: 1

Test case #4/CPU:
build/intel64/Release/smart_classroom_demo_gapi -d_act CPU -d_fd CPU -d_lm CPU -d_reid CPU -i /tmp/tmpqo1jb1xc/data_seq/coco128-subset-480x640x3/input-%04d.jpg -m_act /tmp/tmp57ndv906/models/intel/person-detection-action-recognition-teacher-0002/FP16/person-detection-action-recognition-teacher-0002.xml -m_fd /tmp/tmp57ndv906/models/intel/face-detection-adas-0001/FP16/face-detection-adas-0001.xml -m_lm /tmp/tmp57ndv906/models/intel/landmarks-regression-retail-0009/FP16/landmarks-regression-retail-0009.xml -m_reid /tmp/tmp57ndv906/models/public/Sphereface/FP16/Sphereface.xml -no_show -u cdm

@smirnov-alexey smirnov-alexey mentioned this pull request Dec 6, 2023
Merged
6 tasks
@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Dec 6, 2023

Should be green once OpenCV PR is merged

@smirnov-alexey smirnov-alexey force-pushed the as/fix_gapi_smart_classroom_demo branch from e0fb415 to 3b25938 Compare December 18, 2023 13:03
Wovchena
Wovchena reviewed Dec 18, 2023
View reviewed changes
@Wovchena
Copy link
Copy Markdown
Collaborator

Wovchena commented Dec 19, 2023

I suppose you are OK if this patch isn't included into 23.3 release because it conditionally disables compilation of all GAPI demos just to fix two of them. So I'm not going to rush updating openvino submodule, moreover demos aren't included in packages anyway, so this is more like a formal process.

@Wovchena Wovchena merged commit 9203199 into openvinotoolkit:master Dec 19, 2023
@smirnov-alexey
Copy link
Copy Markdown
Contributor Author

smirnov-alexey commented Dec 19, 2023

@Wovchena awesome, thank you so much! Sure, we can discuss 23.3 release later. Probably, we will convert other demos to OV 2.0 backend and will prepare additional changes to enable demo validation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Wovchena Wovchena Wovchena approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@smirnov-alexey @dmatveev @Wovchena