mirror of
https://github.com/opencv/opencv.git
synced 2024-12-14 08:59:11 +08:00
60848519b5
G-API: Fix incorrect OpaqueKind for Kernel outputs #23843 ### Pull Request Readiness Checklist #### Overview The PR is going to fix several problems: 1. Major: `GKernel` doesn't hold `kind` for its outputs. Since `GModelBuilder` traverse graph from outputs to inputs once it reaches any output of the operation it will use its `kind` to create `Data` meta for all operation outputs. Since it essential for `python` to know `GTypeInfo` (which is `shape` and `kind`) it will be confused. Consider this operation: ``` @cv.gapi.op('custom.square_mean', in_types=[cv.GArray.Int], out_types=[cv.GOpaque.Float, cv.GArray.Int]) class GSquareMean: @staticmethod def outMeta(desc): return cv.empty_gopaque_desc(), cv.empty_array_desc() ``` Even though `GOpaque` is `Float`, corresponding metadata might have `Int` kind because it might be taken from `cv.GArray.Int` so it will be a problem if one of the outputs of these operation is graph output because python will cast it to the wrong type based on `Data` meta. 2. Minor: Some of the OpenVINO `IR`'s doesn't any layout information for input. It's usually true only for `IRv10` but since `OpenVINO 2.0` need this information to correctly configure resize we need to put default layout if there no such assigned in `ov::Model`. See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request - [ ] I agree to contribute to the project under Apache 2 License. - [ ] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV - [ ] The PR is proposed to the proper branch - [ ] There is a reference to the original bug report and related work - [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable Patch to opencv_extra has the same branch name. - [ ] The feature is well documented and sample code can be built with the project CMake |
||
|---|---|---|
| .. | ||
| gapi_int_backend_tests.cpp | ||
| gapi_int_dynamic_graph.cpp | ||
| gapi_int_executor_tests.cpp | ||
| gapi_int_garg_test.cpp | ||
| gapi_int_gmetaarg_test.cpp | ||
| gapi_int_gmodel_builder_test.cpp | ||
| gapi_int_island_fusion_tests.cpp | ||
| gapi_int_island_tests.cpp | ||
| gapi_int_pattern_matching_test.cpp | ||
| gapi_int_perform_substitution_test.cpp | ||
| gapi_int_proto_tests.cpp | ||
| gapi_int_recompilation_test.cpp | ||
| gapi_int_vectorref_test.cpp | ||
| gapi_transactions_test.cpp | ||