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

Port G-API demos to API2.0 - background_subtraction_demo/cpp_gapi #3903

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DariaMityagina
Copy link
Contributor

Ticket:

103104

Description:

G-API Background Subtraction Demo, Classification Benchmark C++ G-API Demo, G-API Gaze Estimation Demo and other - fail on NPU with the following error: dynamic shape issues

get_shape was called on a descriptor::Tensor with dynamic shape
  • OpenVINO is actively removing API 1.0 from their code
  • Both G-API framework and G-API demos use API 1.0 --> which leads to disruption of certain functionality
    Conclusion: We need to port the G-API demos to API2.0 to make them work.

Affected demos:

  • background_subtraction_demo
    [ ERROR ] OpenCV(4.8.0-dev) ..\modules\gapi\src\backends\ov\govbackend.cpp:799: error: (-215:Assertion failed) cv::util::holds_alternativecv::GMatDesc(input_meta) in function 'cv::gimpl::ov::PrePostProcWrapper::cfgPreProcessing' - WIP

Also needs the following change to be merge to OpenCV repo:

with:
repository: opencv/opencv
path: cache/opencv
name: Clone and checkout OpenCV
Copy link
Contributor Author

@DariaMityagina DariaMityagina Jan 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To test this PR in OpenCV : opencv/opencv#24938.
Will be removed.

Copy link
Contributor Author

@DariaMityagina DariaMityagina Apr 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To also test this PR in OpenCV : opencv/opencv#25508.
Will be removed.

@DariaMityagina DariaMityagina force-pushed the icv/dm/background_subtraction_demo_to_OV20 branch 10 times, most recently from c4f5c76 to e036f72 Compare February 2, 2024 10:34
@DariaMityagina DariaMityagina marked this pull request as ready for review February 2, 2024 10:55
@DariaMityagina
Copy link
Contributor Author

This PR should be merged after this one opencv/opencv#24938.

@DariaMityagina DariaMityagina marked this pull request as draft February 19, 2024 17:20
@DariaMityagina DariaMityagina force-pushed the icv/dm/background_subtraction_demo_to_OV20 branch 5 times, most recently from 54f13fe to 23c8a75 Compare April 29, 2024 12:23
@DariaMityagina DariaMityagina force-pushed the icv/dm/background_subtraction_demo_to_OV20 branch from 23c8a75 to 0049139 Compare April 29, 2024 12:37
@@ -241,6 +241,8 @@ int main(int argc, char* argv[]) {
} else {
presenter.handleKey(key);
}
} else {
cv::imwrite("./background-substr-out.jpg", output);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will be removed.

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

Successfully merging this pull request may close these issues.

None yet

1 participant